Repository navigation
extmod/modbluetooth: Add gap_unpair command. - #7845
Open
andrewleech wants to merge 1 commit into
Open
andrewleech wants to merge 1 commit into
andrewleech wants to merge 1 commit into
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #7845 +/- ##
==========================================
- Coverage 98.46% 98.46% -0.01%
==========================================
Files 176 176
Lines 22784 22784
==========================================
- Hits 22435 22434 -1
- Misses 349 350 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
andrewleech
force-pushed
the
unpair
branch
2 times, most recently
from
September 27, 2021 07:57
de857fc to
ae8b7f9
Compare
andrewleech
changed the title
andrewleech
force-pushed
the
unpair
branch
2 times, most recently
from
August 24, 2022 10:10
97e6a24 to
ccb33c3
Compare
RetiredWizard
pushed a commit
to RetiredWizard/micropython
that referenced
this pull request
Apr 16, 2023
This was referenced Feb 29, 2024
Contributor
|
This is an automated heads-up that we've just merged a Pull Request See #13763 A search suggests this PR might apply the STATIC macro to some C code. If it Although this is an automated message, feel free to @-reply to me directly if |
andrewleech
force-pushed
the
unpair
branch
2 times, most recently
from
May 9, 2025 00:22
01fc245 to
0999f14
Compare
|
Code size report: |
andrewleech
force-pushed
the
unpair
branch
2 times, most recently
from
May 27, 2025 05:55
d47ac80 to
6d2bb53
Compare
andrewleech
force-pushed
the
unpair
branch
3 times, most recently
from
September 24, 2025 12:32
33bc83a to
5ff3523
Compare
andrewleech
force-pushed
the
unpair
branch
2 times, most recently
from
November 2, 2025 23:53
ade5914 to
8ef06f7
Compare
Contributor
Author
|
Added Testing:
The test captures bond keys from |
This function removes pairing/bonding information from the bond database, freeing up IRK slots. Implemented for both NimBLE and BTstack backends. Includes multi_bluetooth test and documentation. Signed-off-by: Andrew Leech <[email protected]> Signed-off-by: Andrew Leech <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This function deletes the pairing details from the DB on both the application and the radio, so can be used to free up IRK slots on the radio.
Note: this only works on nimble as pairing/bonding has not been fully implemented on other stacks.
Background
In my application, we have a stm327f765 chip running micropython, with a build of the official nimble uart/hci application running on a separate nrf52810 (https://github.com/apache/mynewt-nimble/tree/master/apps/blehci)
We use this with aioble and paring/bonding.
It's been found that this nimble hci radio only has buffer/storage for 4 RPA / IRK addressing lookups, so if you've got more devices than this stored in the pairing/bonding database, any new devices that come along and try to pair/bond seem to work initially, but upon disconnect/reconnect the pairing details are lost.
Also, if you've got more devices than this stored in the bond database, with the current inplementation of aioble the fact these are stored in a dict/json file means the order is not strictly controlled, so you can't know for certain which devices keys will be loaded first/last into the radio - which ever ones don't fit can fail to connect correctly.
As part of work to provide some control over this issue by allowing an application to limit the number of devices bonded, this gap_unpair() function is needed to remove keys from the ble stack / radio.