search

Aietes / node-red-contrib-harmony

Node-RED integration for Harmony hub
MIT License
15 stars 18 forks source link

MAX_CLIENTS=6 ERROR #4

Closed lespatots closed 6 years ago

lespatots commented 7 years ago

I think this suffers from the problem reported here swissmanu/harmonyhubjs-client and affecting Homebridge as well as Node-Red. The error crashes Node-Red!!

Aietes commented 7 years ago

Unfortunately I currently see no way to catch or prevent this error on my side. I think it's even deeper than swissmanus library, as the exception comes from an underlying library.

lespatots commented 7 years ago

I am sure you are right. I raised the issue more as a warning at this time. Whenever this is fixed, I guess it needs testing. Strangely this problem does not seem to affect Home Assistant!

On 28 Apr 2017, at 08:23, Stefan notifications@github.com wrote:

Unfortunately I currently see no way to catch or prevent this error on my side. I think it's even deeper than swissmanus library, as the exception comes from an underlying library.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Aietes/node-red-contrib-harmony/issues/4#issuecomment-297917504, or mute the thread https://github.com/notifications/unsubscribe-auth/AK2Y3JVWA7k-AvWPN4Vc7YIQYWGzI5--ks5r0YXSgaJpZM4NKhBi.

guevara777 commented 7 years ago

Hi. Great work. Big Thanks to the developer.

Is there any progress on this issue? My Node-Red Setup keeps crashing after 1-2 days and i have to restart it manually.

Bodge-IT commented 7 years ago

I'm getting this too, destabilising my NR. When the crash occurs I get errors from js code in this location: /.node-red/node_modules/node-red-contrib-harmony/node_modules/harmonyhubjs-client/node_modules/node-xmpp-client/node_modules/node-xmpp-core/ Seems to be installing stuff several layers deep, is that correct? Will these nodes themselves be updated with npm outdated?

K-RAD commented 6 years ago

Same issue here, but it crashes my node-red very often when I try to deploy

Aietes commented 6 years ago

This is due to an underlying npm package. I'll soon replace that dependency with an updated library, which will hopefully solve this issue.

lespatots commented 6 years ago

That would be just brilliant, so many people trip over this error.

On 6 Aug 2018, at 11:50, Stefan notifications@github.com wrote:

This is due to an underlying npm package. I'll soon replace that dependency with an updated library, which will hopefully solve this issue.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Aietes/node-red-contrib-harmony/issues/4#issuecomment-410652681, or mute the thread https://github.com/notifications/unsubscribe-auth/AK2Y3GyNmLRHAyVGkFGPAyrCIxZec3xVks5uOBFPgaJpZM4NKhBi.

K-RAD commented 6 years ago

Yeah, I had to remove the plugin.. Until this is ready, is there any way to start an activity by url POST? :)

lespatots commented 6 years ago

Not something I’m familiar with. I would normally trigger an activity through MQTT? Would that work for you?

On 6 Aug 2018, at 18:36, K-RAD notifications@github.com wrote:

Yeah, I had to remove the plugin.. Until this is ready, is there any way to start an activity by url POST? :)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Aietes/node-red-contrib-harmony/issues/4#issuecomment-410770330, or mute the thread https://github.com/notifications/unsubscribe-auth/AK2Y3Lih0YtR2lrGP5sBJ6fqtiFLtp3Sks5uOHCigaJpZM4NKhBi.

Aietes commented 6 years ago

I committed a new version, which is using Airborne's new and improved @harmonyhub libraries. This should solve this issue, and make the extension stable.

lespatots commented 6 years ago

Hi,

Just installed this. Get an error as follows:

'node-red-contrib-harmony:harmony-server : Error: Cannot find module 'autobind-decorator' (line:1)'

I guess I miss a library?

Richard

On 15 Aug 2018, at 23:34, Stefan notifications@github.com wrote:

I committed a new version, which is using Airborne's new and improved @harmonyhub libraries. This should solve this issue, and make the extension stable.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Aietes/node-red-contrib-harmony/issues/4#issuecomment-413345163, or mute the thread https://github.com/notifications/unsubscribe-auth/AK2Y3LFPs1x2ZdEtiV6mGTeHIkESo-zUks5uRJPygaJpZM4NKhBi.