search

UniversalDevicesInc / polyglot-v2

Polyglot Version 2 for use with the ISY 994i 5.0+
MIT License
30 stars 18 forks source link

V2.2.6 with ecobee fails commands #78

Closed larryllix closed 5 years ago

larryllix commented 5 years ago

After many deletions and reinstalls, polyglot and the ecobee NS finally came up and functioned fully last night. Today, I noticed control programs on my ecobee stats not working again. Full status updates are still coming and tested, by setpoint manipulation on the stats.

Attempting manual controls on the admin console device page results in: the first click = no response, the second click on the same command = "No response" from ISY. repeatable every time. The logs show no commands received but happily updates fields from the stats (2) with reports of success. These arethe same symptoms as I had before my last tear down and rebuild of polyglot 2.2.6.

Ecobee_logs_2019-11-10_200531.zip

PG_debug.log

larryllix commented 5 years ago

OK... zoomed in on the problem. The .env file was used properly in polyglot v2.2.5 but causes problems in v2.2.6, namely it will not accept commands from ISY to the Ecobee NS if the .env file exists. I whittled the .env file down to no comments, and only one line, but any occurrence of the file cause the same problem. All statuses have updated despite any properly formatted lines in the .env file.

full.env.txt short.env.txt

jimboca commented 5 years ago

I don't think it has anything to do with the env file. I was testing 2.2.6 on my RPi and experienced the same issue, @Einstein42 and I tried to debug using my machine but couldn't figure it out, he is going to try and repeat on his end...

larryllix commented 5 years ago

I think it has to do with the "USE_HTTPS=false" line in the .env file. It functioned in v2.2.5 but broke my command comms on v2.2.6 from ISY to polyglot somehow. I tried the file installed and removed several times and it failed when installed and worked and worked when removed every time. Thanks guys!!

jimboca commented 5 years ago

Something in 2.2.6 broke, it's an MQTT issue, I've run for months on 2.2.4 with no issue, and have had issues with 2.2.6 since installing on my RPi a few days ago, @Einstein42 will figure it out.

larryllix commented 5 years ago

I am running using https right now, with no problems that I can detect. It's just a PITA defeating the browser warning every time.

On a side note I found ISY deleted some of my ecobee programs when every line involved the MIA stats. grrrrr.....

jimboca commented 5 years ago

Like I said, nothing changed on my side, been using https always with no cert. Mine worked fine all day yesterday, and quit at 7am this morning. I've never seen ISY delete programs. But please don't use this to discuss random issues on this issue, it's hard enough to figure out what is up as is.

mkohanim commented 5 years ago

Is this only on RPi?

Michel Kohanim CEO

(p) 818.631.0333 http://www.universal-devices.comhttp://www.universal-devices.com/

From: jimboca notifications@github.com Sent: Monday, November 11, 2019 9:31 PM To: UniversalDevicesInc/polyglot-v2 polyglot-v2@noreply.github.com Cc: Subscribed subscribed@noreply.github.com Subject: Re: [UniversalDevicesInc/polyglot-v2] V2.2.6 with ecobee fails commands (#78)

Like I said, nothing changed on my side, been using https always with no cert. Mine worked fine all day yesterday, and quit at 7am this morning. I've never seen ISY delete programs. But please don't use this to discuss random issues on this issue, it's hard enough to figure out what is up as is.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/UniversalDevicesInc/polyglot-v2/issues/78?email_source=notifications&email_token=ADMVJPPIBDHNXZQUY2BVPHTQTI5RRA5CNFSM4JLQQGHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDZCRMY#issuecomment-552741043, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADMVJPIHBQ7T4KQN4TFFHBDQTI5RRANCNFSM4JLQQGHA.

jimboca commented 5 years ago

Larry is on Polisy and I'm on Rpi, might be different issues but they seem to be the same.

mkohanim commented 5 years ago

Thanks Jimbo. Can any of you please try PGC?

With kind regards,

Michel Kohanim CEO

(p) 818.631.0333 http://www.universal-devices.comhttp://www.universal-devices.com/

From: jimboca notifications@github.com Sent: Tuesday, November 12, 2019 8:05 AM To: UniversalDevicesInc/polyglot-v2 polyglot-v2@noreply.github.com Cc: Michel Kohanim michel@universal-devices.com; Comment comment@noreply.github.com Subject: Re: [UniversalDevicesInc/polyglot-v2] V2.2.6 with ecobee fails commands (#78)

Larry is on Polisy and I'm on Rpi, might be different issues but they seem to be the same.

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/UniversalDevicesInc/polyglot-v2/issues/78?email_source=notifications&email_token=ADMVJPMHGYXTF7AO7ZDHYNTQTLH2DA5CNFSM4JLQQGHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOED2YC5Q#issuecomment-552960374, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADMVJPNEKZKKD4S433YG47DQTLH2DANCNFSM4JLQQGHA.

jimboca commented 5 years ago

I have Ecobee running on PGC & pgtest, both are running fine.

mkohanim commented 5 years ago

So, the issue is when it’s run locally, correct?

With kind regards,

Michel Kohanim CEO

(p) 818.631.0333 http://www.universal-devices.comhttp://www.universal-devices.com/

From: jimboca notifications@github.com Sent: Tuesday, November 12, 2019 8:25 AM To: UniversalDevicesInc/polyglot-v2 polyglot-v2@noreply.github.com Cc: Michel Kohanim michel@universal-devices.com; Comment comment@noreply.github.com Subject: Re: [UniversalDevicesInc/polyglot-v2] V2.2.6 with ecobee fails commands (#78)

I have Ecobee running on PGC & pgtest, both are running fine.

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/UniversalDevicesInc/polyglot-v2/issues/78?email_source=notifications&email_token=ADMVJPIS5E4MOI5WRIGKYRDQTLKFBA5CNFSM4JLQQGHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOED22KOA#issuecomment-552969528, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADMVJPNDZ3OIRBVZMEDX3NTQTLKFBANCNFSM4JLQQGHA.

jimboca commented 5 years ago

I've been running Ecobee locally on 2.2.6 that came with the Polisy I received Saturday and I've had no issue. It has a .env with only the one line NODE_ENV=development

mkohanim commented 5 years ago

So, this is RPi only?

With kind regards,

Michel Kohanim CEO

(p) 818.631.0333 http://www.universal-devices.comhttp://www.universal-devices.com/

From: jimboca notifications@github.com Sent: Monday, November 18, 2019 10:38 AM To: UniversalDevicesInc/polyglot-v2 polyglot-v2@noreply.github.com Cc: Michel Kohanim michel@universal-devices.com; Comment comment@noreply.github.com Subject: Re: [UniversalDevicesInc/polyglot-v2] V2.2.6 with ecobee fails commands (#78)

I've been running Ecobee locally on 2.2.6 that came with the Polisy I received Saturday and I've had no issue. It has a .env with only the one line NODE_ENV=development

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/UniversalDevicesInc/polyglot-v2/issues/78?email_source=notifications&email_token=ADMVJPPCNQUIXJCVMBJJDHLQULOIBA5CNFSM4JLQQGHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEELO2FA#issuecomment-555150612, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADMVJPMFRW32MJEZKS6B5JTQULOIBANCNFSM4JLQQGHA.

jimboca commented 5 years ago

No @larryllix was on Polisy and did the manual beta upgrade to 2.2.6, I'm on Polisy but with the latest shipped version 2.2.6.

larryllix commented 5 years ago

@jimboca It would possibly help if you tried the line "USE_HTTPS=false" line in the .env file, to see if that is the offender. It worked before the upgrade on v2.2.4 IIRC I have only tried the .env file as a minimum with that line, where polyglot fails, and no .env file at all. where polyglot works fine. Never with any other line.

jimboca commented 5 years ago

@larryllix will do. So you keep the port on 443 and connect with http://polisy:443 or did you change the default port to 80?

larryllix commented 5 years ago

@jimboca Correct. I could never manage to change the port to 80. When I reported problems somebody else forum posted that port 80 would not work. It was one of the first lines to get commented out in .env

I was connecting with http://192.168.0.165:443

At some point in my troubleshooting I got the impression that polyglot used the .env settings and stuffed them in a setup file. Once set they would not change back unless a contrary line was found so the problem created remained until I could find the correct correction line and never did.

mkohanim commented 5 years ago

Nothing below port 443 will work.

With kind regards,

Michel Kohanim CEO

(p) 818.631.0333 http://www.universal-devices.comhttp://www.universal-devices.com/

From: larryllix notifications@github.com Sent: Monday, November 18, 2019 12:37 PM To: UniversalDevicesInc/polyglot-v2 polyglot-v2@noreply.github.com Cc: Michel Kohanim michel@universal-devices.com; Comment comment@noreply.github.com Subject: Re: [UniversalDevicesInc/polyglot-v2] V2.2.6 with ecobee fails commands (#78)

@jimbocahttps://github.com/jimboca Correct. I could never manage to change the port to 80. When I reported problems somebody else forum posted that port 80 would not work. It was one of the first lines to get commented out in .env

I was connecting with http://192.168.0.165:443

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/UniversalDevicesInc/polyglot-v2/issues/78?email_source=notifications&email_token=ADMVJPKKHRTU7TCJGY7KQ23QUL4G7A5CNFSM4JLQQGHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEL2NQQ#issuecomment-555198146, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADMVJPLE3NUEQWWQCGREQBDQUL4G7ANCNFSM4JLQQGHA.

jimboca commented 5 years ago

@larryllix did you turn off 'SSL' in the Node Server Configuration on the ISY?

larryllix commented 5 years ago

No. I wasn't aware of that setting

jimboca commented 5 years ago

You told Polyglot to not use SSL, but ISY is still trying to use it. I've asked @Einstein42 if he could consider fixing that automatically.

larryllix commented 5 years ago

OK. I thought you were referring in the ISY admin console. My bad. There is no "use SSL" field in my polyglot. There is a "Use https" checkbox.

jimboca commented 5 years ago

I was referring to ISY AC, under the "Node Server" -> "Configure" menu, you must turn off SSL for the Slots of the Polyglot instance that has USE_HTTPS=false. It's not "Use SSL' it's just an "SSL checkbox.

larryllix commented 5 years ago

OMG. Too many similar definitions. LOL

Yes my SSL option is checked and was never changed by me at any point.

jimboca commented 5 years ago

Yes, as I mentioned, Polyglot turns that on when it creates the NS since it was in that mode, but then when you turn off https it doesn't disable that. I asked @Einstein42 if that can be fixed.

larryllix commented 5 years ago

ahhh.. co-ordination problem between ISY settings and polyglot settings.

larryllix commented 5 years ago

So If I disable "Use SSL" in the ISY A/C NS setup and reinstall the "USE_HTTPS-false" line back into the .env file it should work?

Gotta take wife out for supper but may try that later tonight. 4:35 PM here.

jimboca commented 5 years ago

Yes, it works, I verified.

larryllix commented 5 years ago

Well "that's why you get the big bucks!" LOL Thanks for that one !

larryllix commented 5 years ago

@jimboca One problem. The "USE_HTTPS=false" is documented as changing the browser access port and protocol, and shouldn't be related to ISY comms.

It works just fine now with the line in .env.

This is from the docs on the code page here.

Overrides the Port that Polyglot listens on for its frontend interface.

HOST_PORT='3000'

Overrides the default mode of HTTPS

USE_HTTPS=false

Later a line is revealed to change the ISY comm parms.

@Einstein42 The function of the .env line functioned properly in polyglot v2.2.5 but broke in polyglot v2.2.6.