Alan-Jowett / sonde / 23569245812
82%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 76
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 85.575%. Remained the same
23569245812

push

github

web-flow 
fix: wire up Android LESC verification and document PSK migration (#508)

* fix: wire up Android LESC verification and document PSK migration (#455)

Wire up \AndroidBleTransport::pairing_method()\ to report the actual
BLE pairing method observed during connection, instead of always
returning \None\.

Java (\BleHelper\):
- Register a \BroadcastReceiver\ for \ACTION_PAIRING_REQUEST\ before
  \createBond()\ to capture \EXTRA_PAIRING_VARIANT\.
- Add \getPairingMethod()\ returning 1 (NumericComparison),
  2 (JustWorks), or 0 (Unknown).
- Unregister the receiver in \disconnectInner()\.

Rust (\AndroidBleTransport\):
- Add \AtomicU8\ to \JniState\ to store the pairing method.
- After successful \connect()\, call \getPairingMethod()\ via JNI and
  store the result.
- \pairing_method()\ now returns \Some(NumericComparison)\,
  \Some(JustWorks)\, or \Some(Unknown)\ — never \None\, since
  Android does not enforce LESC at the OS level (fail-secure).

Docs (\le-pairing-tool-design.md\):
- Update §9.2 to describe the Android LESC verification mechanism.
- Enhance §7.3 plaintext-to-encrypted PSK migration section with
  error cases (\StoreLoadFailed\, \StoreCorrupted\) and crash safety.

Closes #455

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: handle getPairingMethod failure gracefully and clamp JNI value

Address review feedback:
- If \getPairingMethod()\ JNI call fails after a successful
  \connect()\, treat the pairing method as Unknown instead of
  propagating the error (avoids leaking GATT connections).
- Clamp the i32 JNI return value to the valid 0..=2 range before
  storing as u8 to prevent lossy/wrapping casts.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* docs: update MTU value to match BleHelper implementation (517)

The design doc §9.2 stated requestMtu(247) but BleHelper.connect()
actually calls requestMtu(517). Updated to match the implementation.

Co-authored-by: Copilot <223556219+Copilot... (continued)

22164 of 25900 relevant lines covered (85.58%)

162.92 hits per line

Jobs
ID Job ID Ran Files Coverage
1 23569245812.1 76
85.58
 GitHub Action Run
Source Files on build 23569245812