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

homebridge crashing when broadlnik rm module is present in config.json #410

Closed
dev0x451 opened this issue Jul 24, 2018 · 3 comments
Closed
Labels

Comments

@dev0x451
Copy link

dev0x451 commented Jul 24, 2018

Hi, whenever I add to my config file this portion to try broadlink rm the whole homebridge crashes.
I'm running Win 10 64 Pro. The other modules work fine...
The part of my config.json:

"platforms": [{
"platform": "BroadlinkRM",
"name": "Broadlink RM",
"hideScanFrequencyButton": false,
"hideLearnButton": false,
"homebridgeDirectory": "/Users/Admin/.homebridge",
"accessories": [{
"name": "TV On/Off",
"type": "switch",
"data": {

		}
	}]
}]

The crash log:

[2018-7-24 23:35:12] [Broadlink RM] **************************************************************************************************************
[2018-7-24 23:35:13] [Broadlink RM]
C:\Users\Admin\AppData\Roaming\npm\node_modules\homebridge-broadlink-rm\node_modules\ping\lib\parser\win.js:56
this._times.push(parseFloat(match[1], 10));
^

TypeError: Cannot read property '1' of null
at WinParser.processBody (C:\Users\Admin\AppData\Roaming\npm\node_modules\homebridge-broadlink-rm\node_modules\ping\lib\parser\win.js:56:42)
at WinParser.parser.eat (C:\Users\Admin\AppData\Roaming\npm\node_modules\homebridge-broadlink-rm\node_modules\ping\lib\parser\base.js:125:14)
at C:\Users\Admin\AppData\Roaming\npm\node_modules\homebridge-broadlink-rm\node_modules\underscore\underscore.js:76:21
at Function.
.each._.forEach (C:\Users\Admin\AppData\Roaming\npm\node_modules\homebridge-broadlink-rm\node_modules\underscore\underscore.js:186:9)
at ChildProcess. (C:\Users\Admin\AppData\Roaming\npm\node_modules\homebridge-broadlink-rm\node_modules\ping\lib\ping-promise.js:76:12)
at Object.onceWrapper (events.js:272:13)
at ChildProcess.emit (events.js:180:13)
at maybeClose (internal/child_process.js:936:16)
at Process.ChildProcess._handle.onexit (internal/child_process.js:220:5)

C:\Users\Admin\AppData\Roaming\npm>

@tomdrumdrum
Copy link

Did you solve your problem and how? I have the same issue.

@dev0x451
Copy link
Author

dev0x451 commented Jan 31, 2019 via email

@lprhodes lprhodes added the bug label Jan 1, 2020
@lprhodes
Copy link
Owner

lprhodes commented Jan 1, 2020

As of 3.6.11 I'm using a new ping module, hopefully this resolves your issue.

@lprhodes lprhodes closed this as completed Jan 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants