-
Notifications
You must be signed in to change notification settings - Fork 0
add authentication support #129
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
Conversation
8cae448 to
1ccf7e2
Compare
Kconfig
Outdated
| help | ||
| Filter scanned devices by bonded status. Use it for authenticated | ||
| connections, so that discovered devices will be ignored when not bonded | ||
| already. Bonding is can be triggered explicitly by calling |
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.
| already. Bonding is can be triggered explicitly by calling | |
| already. Bonding can be triggered explicitly by calling |
| return; | ||
| } | ||
|
|
||
| if (tf.is_bonded && !sync_requested(&tf.adv_data)) |
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.
Nice catch on this corner case!
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.
Using this file is optional - if an application is using custom scan logic, then it can also have explicit control over bonding, correct?
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.
Yes, that is right, it is just a "utility". But it is still useful, which is why I reused it for samples/custom_connect/.
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.
Great! It might make sense to reorganize the library a bit so scan.c and bond.c are more obviously utilities, but for now I think this is great.
sam-golioth
left a comment
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.
🚀
Signed-off-by: Marcin Niestroj <[email protected]>
Introduce a new module for controlling Bluetooth bonding (pairing) on the gateway. The module provides a simple API for enabling and disabling bonding with configurable timeout support. The API supports time-limited bonding windows where bonding can be enabled for a specific duration and automatically disabled afterwards. Signed-off-by: Marcin Niestroj <[email protected]>
Add support for filtering scanned BLE devices based on their bonding status. When POUCH_GATEWAY_GATT_SCAN_FILTER_BONDED is enabled, the gateway only connects to devices that are already bonded or when bonding is explicitly enabled via the bonding API. This enables authenticated connections where devices must be paired before they can communicate with the gateway. Signed-off-by: Marcin Niestroj <[email protected]>
Integrate bonding control into the gateway application with button-triggered pairing and passkey confirmation support. Enable 30 second window for bonding after first button press. Confirm passkey using second button press, during that window. Signed-off-by: Marcin Niestroj <[email protected]>
1ccf7e2 to
3d73fa9
Compare
Integrate bonding control into the gateway application with
button-triggered pairing and passkey confirmation support.
Enable 30 second window for bonding after first button press. Confirm
passkey using second button press, during that window.