Skip to content

feat: adding new modes to gappairing for future enhancements in secureconnections #912

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.

Sign up for GitHub

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

Open
wants to merge 18 commits into
base: main
Choose a base branch
from
Open

feat: adding new modes to gappairing for future enhancements in secureconnections #912

Changes from 14 commits
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
a417192
Work towards improving pairing security requirements
XavierBernhardt-Philips Jun 2, 2025
645ebe1
Better enum naming for ConnectionMode and ManInTheMiddleMode
MiquelJayson-Philips Jun 6, 2025
f470d63
Renamed notRequired to Disabled
MiquelJayson-Philips Jun 6, 2025
f3e8678
Rename values in SecureConnectionMode enum
MiquelJayson-Philips Jun 13, 2025
3adc019
Refactoring SecureConnectionMode and MITMMODE into 2 functions
XavierBernhardt-Philips Jun 16, 2025
5afb95c
refactoring
XavierBernhardt-Philips Jun 18, 2025
1e16bc3
Merge branch 'main' into feature/pairing-security-requirements
XavierBernhardt-Philips Jun 18, 2025
1459c1d
fixed compile error
XavierBernhardt-Philips Jun 18, 2025
4199635
Removed bracket
MiquelJayson-Philips Jun 19, 2025
6cad9a7
Merge commit 'c20c83ea909ca2e1a7ef8a07bb5cfe91144bf3a7'
XavierBernhardt-Philips Jul 2, 2025
76a781c
revert changes to gap
MiquelJayson-Philips Jul 14, 2025
18f12a1
Merge branch 'main' into feature/156620_revert_gap_sc
MiquelJayson-Philips Jul 16, 2025
d6edad6
cleanup
MiquelJayson-Philips Jul 17, 2025
eb79daa
cleanup
MiquelJayson-Philips Jul 17, 2025
ebeec25
pr comments
MiquelJayson-Philips Jul 24, 2025
143ec43
Update SetSecurityMode to include new SC and MITM
MiquelJayson-Philips Jul 24, 2025
6da3197
update mock
MiquelJayson-Philips Jul 24, 2025
330a9aa
add not supported mitm mode
MiquelJayson-Philips Jul 24, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 14 additions & 3 deletions services/ble/Gap.hpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -128,13 +128,24 @@ namespace services
level4,
};

virtual void Pair() = 0;
// Values align with STM32 sdk ble core defs
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think we should align with a specific manufacturer. if for some reason the manufacturer has a different values, the class implement this interface should be responsible for adapting it.

enum class SecureConnectionMode : uint8_t
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we have the same three options for mitm and secureconnections:
notSupported,
supported,
enforced,

For some vendors, such as ST, we might not have the implementation for all combinations, which is fine.

{
notSupported,
supported,
enforced,
};

virtual void AllowPairing(bool allow) = 0;
enum class ManInTheMiddleMode : uint8_t
{
disabled,
enforced,
};

virtual void Pair() = 0;
virtual void AllowPairing(bool allow) = 0;
virtual void SetSecurityMode(SecurityMode mode, SecurityLevel level) = 0;
virtual void SetIoCapabilities(IoCapabilities caps) = 0;

virtual void AuthenticateWithPasskey(uint32_t passkey) = 0;
virtual void NumericComparisonConfirm(bool accept) = 0;
};
Expand Down
Loading