-
-
Notifications
You must be signed in to change notification settings - Fork 8
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
Lookup devices fails #57
Comments
Can you confirm Node-RED version you are running? |
node-red 0.20.2 |
Can you confirm if this issue persists with Node-RED 0.20.3 (currently running this myself, with the latest version of the nodes) |
The issue persists with Node-RED 0.20.3 |
Thanks, I can't replicate on my production instance. Going to build a fresh Node-RED install and test. |
node-red-contrib-alexa-smart-home updated to 0.4.30 / published on npmjs, can you advise if this version shows the same issue. |
Now my Node-RED container restarts regularly. The log shows the following error: [error] [alexa-smart-home-v3-conf:e47a58f4.a050c8] TypeError: First argument should be a valid curve name And a lot of this: [warn] [alexa-smart-home-v3:device name] Unable to register device node, account not configured! |
That's not good :( Can you confirm how you've installed Node-RED, and importantl;y what version of node.js you are running, when Node-RED starts you'll see some thing like this in the Node-RED log:
I suspect it is Node related, rather than Node-RED/ the update itself. |
Based upon this assumption, I've modified a revision of the nodes to only apply the 'ecdhCurve' request parameter for node.js 8 deployments or newer. This parameter was added fairly recently to address this known issue. @nordish1, please do let me know the NodeJS version you're running to confirm this is root cause (or discount). Sorry for any inconvenience. |
Did this resolve your issue? |
@coldfire84 Sorry, but I can't check it before weekend |
+1 - same issue Node Red: V0.20.3 are you saying we need to downgrade to node v8.15? |
Can you share full console logs please. |
Crashing for me also,
Quick and dirty fix ;-) |
Update: after several issues and Tests, some feedback.
#After adding one or more or edit feature service crash / loop Welcome to Node-RED Your flow credentials file is encrypted using a system-generated key. 28 Mar 12:36:18 - [info] Starting flows 28 Mar 12:37:29 - [info] Node-RED version: v0.20.3 Your flow credentials file is encrypted using a system-generated key. 28 Mar 12:37:35 - [info] Starting flows 28 Mar 12:38:46 - [info] Node-RED version: v0.20.3 Your flow credentials file is encrypted using a system-generated key. 28 Mar 12:38:52 - [info] Starting flows 28 Mar 12:38:55 - [info] Node-RED version: v0.20.3 Your flow credentials file is encrypted using a system-generated key. Welcome to Node-RED 28 Mar 13:04:36 - [info] Node-RED version: v0.20.3 Your flow credentials file is encrypted using a system-generated key. 28 Mar 13:04:42 - [info] Starting flows |
Thanks - I have now replicated - there are two sets of changes required:
And the changes mentioned by @RobertHerter . I am testing these now and will confirm once working npm released. |
You can now test with version 0.4.42 |
with 0.4.42 [node-red-contrib-alexa-smart-home/alexa] Error: Cannot find module 'semver' (line:24) |
Thanks for this - this was a result of me moving away from "request" library to "https" - I've integrated your changes. "Request" library continuously generates the TLS/ curve errors. |
@coldfire84 on v4.38 - seems solved now, can add multiple devices / features without crash - will test further. Is v4.42 npm or via pallet ? |
@RobertHerter I have set a depends on semver 6.0.0 in version 0.4.43 (are you using nodered "lite" install? My fresh install of nodered docker doesn't generate this issue). |
@coldfire84 using nodered on raspi installed via npm, installing semver globaly fixed it, lookes like missing in dependencies |
When I update the alexa-smart-home-v3-node to search for devices, no devices will be found.
The log shows:
Problem looking up unsername's devices
,getDevices error: TypeError: First argument should be a valid curve name
The text was updated successfully, but these errors were encountered: