search

yadomi / node-red-contrib-philipshue-events

Implements the Philips Hue API v2 EventSource as a node-red node
MIT License
13 stars 2 forks source link

Deploying multiple times breaks the payload #4

Closed sockless-coding closed 2 years ago

sockless-coding commented 2 years ago

Deploying the flow multiple times breaks the payload object, instead of returning an actual JSON object it returns [object Object]. The only way I managed to get it to work again is to reload node-red.

yadomi commented 2 years ago

Thanks for the report, I encountered this error as well. I think it's because I'm not handling the disconnect of the EventSource correctly.

I'll look at this soon

yadomi commented 2 years ago

@sockless-coding do you still encounter the issue ? Connection errors should now be handled (see https://github.com/yadomi/node-red-contrib-philipshue-events/commit/8a60417bff5e21ef1ffcdb1b3a7f37e70855d5fe, since v1.2.0), but there may be some edge case that I didn't catch yet.

yadomi commented 2 years ago

I'm closing since I've no longer encountered the issue since 1.2.0. Feel free to reopen if you have the issue again