Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

There is a possible mistake in emulate.js #14

Closed
AlGu1 opened this issue Feb 9, 2018 · 7 comments
Closed

There is a possible mistake in emulate.js #14

AlGu1 opened this issue Feb 9, 2018 · 7 comments

Comments

@AlGu1
Copy link

AlGu1 commented Feb 9, 2018

Hi,

i am using this Lib by biddster/node-red-contrib-wemo-emulator. And die node-red in my ioBroker to emulate devices from my homeautomation for Amazon Alexa.
I have installed the lib "node-red-contrib-wemo-emulator" in my node-red and using it over Alexa speech control, that works fine. But in the Alexa App i can not control the device. It shows "Belkin International Inc." and loading icon. But when i change the row 178 in emulate.js into:
UDN: "uuid:" + this.serial
then can i control the device in the Alexa App.

I think it is a bug, but i can not check it in your using of this lib.

Thanks
AlGu

@dhleong
Copy link
Owner

dhleong commented Feb 9, 2018

Have you tried discussing your issue there? I don't use that lib, but use this one directly and don't have any problems with Alexa.

@AlGu1
Copy link
Author

AlGu1 commented Feb 9, 2018

Ok mhh...but is the UDN with "uuid:" not the correct code?! Other emulations like node-red-contrib-alexa-local use it so.
Greetings
AlGu

@dhleong
Copy link
Owner

dhleong commented Feb 9, 2018

It seems to work for me without it. I would want a proper UUID if I'm going to prefix with uuid:; the serial number we're generating is definitely not a UUID.

jtai added a commit to jtai/wemore that referenced this issue Mar 4, 2018
@jtai
Copy link

jtai commented Mar 4, 2018

FWIW I ran into the same issue as @AlGu1 using wemore directly. I was previously using https://github.com/kakopappa/arduino-esp8266-alexa-multiple-wemo-switch to emulate switches, which worked just fine. When I switched to wemore the alexa app hung as described above. Changing the UDN to prefix with uuid: made it work.

From reading through issues like kakopappa/arduino-esp8266-alexa-multiple-wemo-switch#22, it sounds like different versions of echo hardware behave differently, which may account for the difference in experiences?

@dhleong
Copy link
Owner

dhleong commented Mar 4, 2018

Thanks for the report @jtai, that's very interesting. Would you mind testing with generating an actual UUID to be used after uuid: with something like UUID v4? If that still works for you then I'll be happy to verify that it still works on my setup and, assuming it does, implement it into the library.

jtai added a commit to jtai/wemore that referenced this issue Mar 4, 2018
@jtai
Copy link

jtai commented Mar 4, 2018

Alexa seems ok with the UUID, at least on my echo. This works: jtai@cd374c6

@dhleong dhleong closed this as completed in 19c5997 Mar 5, 2018
@dhleong
Copy link
Owner

dhleong commented Mar 5, 2018

Hello everyone, thanks for your help here! I've released version 0.6.0 that uses the uuid: prefix. Please give it a try and let me know if it works for you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants