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

fix: Advertised names are not showing #11

Open
jefflongo opened this issue Oct 3, 2024 · 2 comments
Open

fix: Advertised names are not showing #11

jefflongo opened this issue Oct 3, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@jefflongo
Copy link
Contributor

Description

Advertised names are empty strings.

Steps To Reproduce

  1. Scan for devices
  2. Observe name is empty string

Expected Behavior

Expected to see non-empty name. Works on other platforms, and Windows itself can see the peripheral name.

Screenshots

N/A

Additional Context

I believe this to be a problem in WinBLE, since WinBLE is reporting the name as an empty string.

@jefflongo jefflongo added the bug Something isn't working label Oct 3, 2024
@chan150
Copy link
Owner

chan150 commented Oct 4, 2024

Hi @jefflongo
It seem to be WinBLE-side problem as you guess.
I check WinBLE library to fix issues if i can.

@jefflongo
Copy link
Contributor Author

jefflongo commented Oct 10, 2024

I've dug into this issue a bit, and have determined the issue. WinBLE receives two types of scan results:

  1. advType = 'ScanResponse': this result contains the name but not the advertised services.
  2. advType = 'ConnectableUndirected': this result contains advertised services but not the name.

Full documentation on the advTypes here.

So I don't think it's possible to fully construct the BluetoothDeviceWindows (or at least capture both the name and advertisement data) in the scan result stream. Or perhaps the addOrUpdate can fill in the missing data when it comes, instead of replacing the entire object.

Separately, FBP Windows needs an advName getter. It can just be redirected to platformName since they are the same.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants