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

Typeerror when using Peripheral.readHandle #22

Closed
tal opened this issue Jan 23, 2019 · 4 comments
Closed

Typeerror when using Peripheral.readHandle #22

tal opened this issue Jan 23, 2019 · 4 comments

Comments

@tal
Copy link

tal commented Jan 23, 2019

When I try and call Peripheral.readHandle I get the following error:

TypeError: There should be 4 arguments: (String, String, String, String)
 at Noble.readHandle (/Users/atlastal/Projects/bt-explore/node_modules/noble/lib/noble.js:429:18)
 at Peripheral.readHandle (/Users/atlastal/Projects/bt-explore/node_modules/noble/lib/peripheral.js:120:15)
 at p.connect (/Users/atlastal/Projects/bt-explore/dist/index.js:15:19)
 at Peripheral.<anonymous> (/Users/atlastal/Projects/bt-explore/node_modules/noble/lib/peripheral.js:38:7)
 at Object.onceWrapper (events.js:315:30)
 at emitOne (events.js:116:13)
 at Peripheral.emit (events.js:211:7)
 at Noble.onConnect (/Users/atlastal/Projects/bt-explore/node_modules/noble/lib/noble.js:191:16)
 at emitTwo (events.js:126:13)
 at NobleMac.emit (events.js:214:7)
@tal
Copy link
Author

tal commented Jan 23, 2019

ah sorry, his is an issue with https://github.com/noble/noble

@tal tal closed this as completed Jan 23, 2019
@tal
Copy link
Author

tal commented Jan 23, 2019

Actually I take it back. it appears as though it's an issue with the handle. Not sure if that's the mac or root noble. Any thoughts?

https://github.com/noble/noble/blob/master/lib/noble.js#L428-L430

@tal tal reopened this Jan 23, 2019
@geovie
Copy link
Collaborator

geovie commented Jan 23, 2019

This is a bug in noble-mac, I will fix it soon.
Thanks for reporting!

@geovie
Copy link
Collaborator

geovie commented Feb 15, 2019

Fixed, Thanks

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

2 participants