Skip to content
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

Abstract Chip-tool #15

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Abstract Chip-tool #15

wants to merge 4 commits into from

Conversation

mleisner
Copy link
Owner

Problem

Example chip-tool currently does not have a flexible way to add external/third-party PKIs

Change overview

  • chip-tool now builds the core as a static library (chip-tool-utils)
  • Added set of pure virtual methods PKI-related to Commands class
  • Added example implementation of Matter’s example PKI
  • Added DAC to GenerateNOCChain method call
  • Added GenerateNOCSR method to OperationalCredentialsDelegate class so every PKI-vendor can override the nonce generation to follow a custom set of rules. Random example added.

Testing

  • Tested Commissioning flow (chip-tool vs lighting-app)
  • Matter Unit Tests

@mleisner mleisner changed the title chip-tool now builds the core as a static library (chip-tool-utils) Abstract Chip-tool Oct 13, 2021
Added set of pure virtual methods PKI-related to Commands class

Added example implementation of Matter’s example PKI

Added DAC to GenerateNOCChain method call

Added GenerateNOCSR method to OperationalCredentialsDelegate class so every PKI-vendor can override the nonce generation to follow a custom set of rules. Random example added.
@mleisner mleisner force-pushed the feature/abstract-chiptool branch from fcaf440 to dafba69 Compare October 13, 2021 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant