Skip to content
Snippets Groups Projects

New pairing database

Merged schneider requested to merge schneider/bonding-db into master

Goal is to write individual pairings (bonds) to separate file and have a more flexible file format. Currently all pairings (bonds) are stored in a single file (pairings.bin).

Goal is to store each pairing (bond) in a separate file in a pairings/ directory, named pairingN.bin (with N being 1,2,3,...). Each file stores a list of TLVs and has the version of the file format as first byte.

What I still want to implement:

  • Remove ble.txt and mac.txt and use card10.cfg instead.

In the future I'd like to be able to read the other device's name after pairing and write that into the file so the user has an idea which device is represented by that pairing.

Edited by schneider

Merge request reports

Merge request pipeline #4728 passed

Merge request pipeline passed for 747803cc

Approval is optional

Merged by rahixrahix 4 years ago (Oct 6, 2020 8:34am UTC)

Merge details

  • Changes merged into master with d788c494.
  • Deleted the source branch.

Pipeline #4729 passed

Pipeline passed for d788c494 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • rahix
  • rahix
  • rahix
  • rahix
  • rahix
  • schneider added 4 commits

    added 4 commits

    • 6d7635d9 - feat(bondings): Write each bond to a separate file
    • 93234db7 - fix(bondings): Bring API more in line with upstream
    • 3e24328f - fix(ble): Generate new ECC keys after pairing
    • 747803cc - change(ble): Use card10.cfg instead of ble.txt and mac.txt

    Compare with previous version

  • Looks good to me! Thanks for keeping our codebase clean.

    However, I'm only a bot - so a human will still have to approve this request.

  • rahix resolved all threads

    resolved all threads

  • rahix approved this merge request

    approved this merge request

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading