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

chip-tool now builds the core as a static library (chip-tool-utils) #18

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

Conversation

mleisner
Copy link
Owner

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.

Problem

What is being fixed? Examples:

Change overview

What's in this PR

Testing

How was this tested? (at least one bullet point required)

  • If unit tests were added, how do they cover this issue?
  • If unit tests existed, how were they fixed/modified to prevent this in future?
  • If new unit tests are not added, why not?
  • If integration tests were added, how do they verify this change?
  • If new integration tests are not added, why not?
  • If manually tested, what platforms controller and device platforms were manually tested, and how?
  • If no testing is required, why not?

Marty Leisner added 2 commits October 14, 2021 20:52
    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/chiptool-lowermerge branch from 788391a to c9010fe Compare October 15, 2021 05:06
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