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

Reported address under Windows 10 is not only address #128

Open
tzvetelin-vassilev opened this issue Dec 6, 2016 · 6 comments
Open

Reported address under Windows 10 is not only address #128

tzvetelin-vassilev opened this issue Dec 6, 2016 · 6 comments

Comments

@tzvetelin-vassilev
Copy link

Under Windows 10 reported address looks wrong. Under Windows 7, Windows 8 and Windows 8.1 field address contains exact address.

Inquiry results:

name: Tzvetelin�s iMac
address: (8C:2D:AA:43:CD:E0)Tzvetelin�s iMac

name: Dragomir�s iMac
address: (10:40:F3:E4:C6:BA)Dragomir�s iMac

name: Wacom Sofia QA Dev SM
address: (20:D3:90:B7:C2:FF)Wacom Sofia QA Dev SM

List paired devices result:

{"name":"Tzvetelin�s iMac","address":"(8C:2D:AA:43:CD:E0)","services":[{"channel":1,"name":"SPP"}]}
{"name":"Wacom Sofia QA Dev (SM","address":"(20:D3:90:B7:C2:FF)","services":[{"channel":1,"name":"SPP"}]}
{"name":"Dragomir�s iMac","address":"(10:40:F3:E4:C6:BA)","services":[{"channel":1,"name":"SPP"}]}

@tzvetelin-vassilev tzvetelin-vassilev changed the title Reported addres under Windows 10 is not only address Reported address under Windows 10 is not only address Dec 6, 2016
@eelcocramer
Copy link
Owner

Hi, thank you for your report.

Are these 2 separate issues? If so please create a separate issue for each issue to keep the discussion clean and on topic. If not can you be more clear on how your comment relates to the issue.

As for your issue on Windows 10. What does the output you put in the issue show? Was it created on Windows 10 or another release? How is it different from another windows version?

I hardly ever use windows but Bluetooth addresses can be formatted differently on different platforms. An address looks different on Linux than on OSX as these use different byte separators.

@eelcocramer
Copy link
Owner

cc: @langholz

@eelcocramer
Copy link
Owner

As for you other comment. If this is another issue, please create one and give some more details about:

  • What it is that make you think the program crashes on that line
  • When you have it available, please include a crash log
  • Details on the version of the module you are using together with the version of your operating system and the version of nodejs you are using.

Thanks!

@tzvetelin-vassilev
Copy link
Author

Hi,

Inquiry results comes from "found" event where address under windows 10 is something like this: (8C:2D:AA:43:CD:E0)Tzvetelin�s iMac
until in previous versions of windows address is: 8C:2D:AA:43:CD:E0

List paired devices comes from listPairedDevices method and address under windows 10 looks like this: (8C:2D:AA:43:CD:E0)
until in previous versions of windows address is: 8C:2D:AA:43:CD:E0

I understand delimiters are different under different operating systems, but result from windows 10 looks like some description, not address only.

For other question I'll move under another issue.

@eelcocramer
Copy link
Owner

Ok, thank you for making this clear to me.

@eelcocramer eelcocramer added the bug label Dec 7, 2016
@eelcocramer
Copy link
Owner

This seems like a bug to me. In this line a discovered Bluetooth socket is converted to a human readable address string. The following statement checks if this conversion is successful and if so this line tries to get the bluetooth address from the human readable string. If that last line fails the human readable string is used and this seems to happen on your Windows 10 setup.

I'm not sure why this is happening and I'm also not able to fix this myself as I'm not familiar with Windows programming and more important I do not have a Windows development environment available.

Hopefully some one else is able to look into this further.

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

No branches or pull requests

2 participants