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

Android: fix 'Descriptor None was not found!' #1580

Merged
merged 3 commits into from
May 23, 2024

Conversation

vabt-igd
Copy link
Contributor

@vabt-igd vabt-igd commented May 23, 2024

Use the 128bit UUID plus the Bluetooth base UUID for the notification descriptor on android devices. This fixes Descriptor None was not found! exceptions, as discussed in #823.

On the three android devices (API Level 33 and 32) I tested, this seems to work reliably.

As this is just a minor fix, I did not add an entry to the CHANGELOG.rst.
Let me know if that should be changed.

This fix may also address #828.

Copy link
Collaborator

@dlech dlech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As this is just a minor fix, I did not add an entry to the CHANGELOG.rst.

Definitely worth adding a changelog entry!

bleak/backends/p4android/defs.py Outdated Show resolved Hide resolved
@vabt-igd vabt-igd force-pushed the fix_android_descriptor_none branch from bee4d86 to 0b042fa Compare May 23, 2024 19:59
bleak/backends/p4android/defs.py Outdated Show resolved Hide resolved
CHANGELOG.rst Outdated Show resolved Hide resolved
@vabt-igd vabt-igd force-pushed the fix_android_descriptor_none branch from 254167d to 6907ead Compare May 23, 2024 21:06
@vabt-igd vabt-igd requested a review from dlech May 23, 2024 21:12
@vabt-igd
Copy link
Contributor Author

I think, this should be it now. Sorry for the mess.
Tested on API 33 and 32 phones using three different services with corresponding characteristics (custom and non-custom) running on NRF52 chips.

CHANGELOG.rst Outdated Show resolved Hide resolved
@dlech dlech merged commit d45ec90 into hbldh:develop May 23, 2024
13 checks passed
@dlech
Copy link
Collaborator

dlech commented May 23, 2024

Thanks for testing again!

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

Successfully merging this pull request may close these issues.

2 participants