-
Notifications
You must be signed in to change notification settings - Fork 3k
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
BLE: Fix ble gattserver autorization list registration #10094
BLE: Fix ble gattserver autorization list registration #10094
Conversation
@desmond-blue, thank you for your changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm afraid that change isn't correct. We need to cache characteristic that send update as we consult the getUpdateSecurityRequirement latter.
65b2a49
to
196a766
Compare
Hi @pan- , got it, then I believe the condition check for update properties should be changed from |
@desmond-blue Any updates on this pr ? |
@pan- I've done some changes on original commit, could you check if it's good? Thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@desmond-blue thanks.
CI started |
Test run: FAILEDSummary: 1 of 13 test jobs failed Failed test jobs:
|
restarted jenkins-ci/mbed-os-ci_greentea-test |
Description
This is for #10028, GattServer maintains an authorization list which is registered by attribute permission check,
UPDATE_PROPERTIES
which equalsNOTIFY_PROPERTY | INDICATE_PROPERTY
is not a correct permission check.Pull request type
Reviewers
Release Notes