-
Notifications
You must be signed in to change notification settings - Fork 611
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
Has 1.7.5 broken 16 bit service UUID's on iOS? #1031
Comments
Also experiencing this issue with versions 1.7.5 and 1.7.6. |
Can confirm the regression. Will get out a fix shortly. |
Additional UUID validation on iOS inadvertantly prevented 16 & 32-bit UUID formats from being accepted.
Additional UUID validation on iOS inadvertantly prevented 16 & 32-bit UUID formats from being accepted.
A fix has been released in 1.7.7 (and I've added some more test cases to cover this!) Thanks for the quick reports! |
Thank you! The main problem is solved. The message "Malformed UUID: 180F" is gone, but ble.startNotification still shows the second message, "The attribute could not be found." |
@gwhenne is this issue on iOS or Android? What you are describing sounds unrelated to this particular regression. |
Thank you. Yes, it does appear to be a different problem. A new issue has been opened here: |
Hi,
The fixes in #1021 for #905 and #1014 seem to have broken 16 bit service UUID's like 180a (Device Information Service). Calling
ble.read
with "180a" as service_uuid used to work fine, but now results in an error sayingMalformed UUID: 180a
This looks like l. 1104 in
BLECentralPlugin.m
, and it matches with the change from using CBUUID'sUUIDWithString
to NSUUID'sinitWithUUIDString
.Using the full 128 bit UUID (like in
UUIDHelper.java
) doesn't work. The UUID is no longer rejected but we end up withCould not find service with UUID 0000180A-[...]
instead; it doesn't seem to match the services reported by the device.
The text was updated successfully, but these errors were encountered: