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

Root certificates for Network Operational Credential (NOC) #524

Closed
ashcherbakov opened this issue Nov 10, 2023 · 3 comments
Closed

Root certificates for Network Operational Credential (NOC) #524

ashcherbakov opened this issue Nov 10, 2023 · 3 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@ashcherbakov
Copy link
Contributor

ashcherbakov commented Nov 10, 2023

Root CA that NOCs chain up to.
It's similar to ROOT_CERT transactions (see https://github.com/zigbee-alliance/distributed-compliance-ledger/blob/master/docs/transactions.md#x509-pki), but they are managed (publish/revoke) by Vendors and do not require Trustee approval.

Acceptance Criteria:

@ashcherbakov ashcherbakov added this to the v1.3: DCL 1.3 milestone Nov 10, 2023
@ashcherbakov ashcherbakov added the enhancement New feature or request label Nov 21, 2023
@ashcherbakov ashcherbakov changed the title Schema for Network Operational Credential (NOC) Root certificates for Network Operational Credential (NOC) Dec 19, 2023
@ashcherbakov
Copy link
Contributor Author

PR with Design: #529
There are open questions: https://github.com/zigbee-alliance/distributed-compliance-ledger/blob/master/docs/design/noc-root-cert-design.md#questions that needs to be discussed on the DCL TT call.

@ashcherbakov
Copy link
Contributor Author

ashcherbakov commented Jan 4, 2024

Discussed the open questions on DCL TT call:

  • Should the vendor add a revocation distribution point for NOC certificates: NO, just revocation/remove (the same as for PAA Roots) is enough
  • Should the following queries return NOC Certificate?
    • GET_ALL_SUBJECT_X509_CERTS: YES
    • GET_ALL_X509_ROOT_CERTS: NO
    • GET_X509_CERT: YES
  • Should an additional field be added to the certificate schema to distinguish NOC certificates from common PAAs/PAIs: YES
  • Should a revoked NOC Root Certificate be stored in the revoked list, or should it be completely removed? TWO separate APIs: Remove (completely remove; needed in case of mistakes) and Revoke
  • if a NOC Root Certificate is revoked, should it be returned in the existing
    • GET_ALL_REVOKED_X509_ROOT_CERTS: NO
    • GET_REVOKED_CERT: YES

Other things to include/update design:

  • NOC certs are not VID-scoped (there is no VID field there)
  • When adding a NOC cert, need to check that there is no certificate with the same Subject+SKID already published by another vendor
  • A possibility to re-add removed NOC certificates
  • Intermediate certs (ICA) signed by NOC: can be published only by the Vendor who published the corresponding NOC
  • Need to update the docs/CLI calls names to distinguish between PAA/PAI and NOC/ICA certs. Need to do it in a compatible way.

Open questions:

  • Revocation distribution point for NOC Roots (to list revoked intermediate certificates signed by this NOC)
  • The current ADD_X509_CERT allows to add an Intermediate/leaf cert by any account as long as the cert is chained back to a PAA (Root) on the ledger. Do we need to keep this API or disable/modify it?

akarabashov added a commit that referenced this issue Feb 20, 2024
…oot-certificate

#524 Implement adding and requesting root NOC certificates
Abdulbois added a commit that referenced this issue Mar 6, 2024
Enable revoking NOC Root certs

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 12, 2024
Enable revoking NOC Root certs

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 12, 2024
Enable revoking NOC Root certs

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 13, 2024
Enable revoking NOC Root certs

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 13, 2024
Enable revoking NOC Root certs

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 13, 2024
Enable revoking NOC Root certs

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 13, 2024
Enable revoking NOC Root certs

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 13, 2024
Enable revoking NOC Root certs

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 13, 2024
Enable revoking NOC Root certs

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 13, 2024
Enable revoking NOC Root certs

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 13, 2024
Enable revoking NOC Root certs

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 13, 2024
Enable revoking NOC Root certs

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 14, 2024
Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 14, 2024
Minor refactoring due to comments of PR

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 14, 2024
Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 14, 2024
Minor refactoring due to comments of PR

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 14, 2024
Minor refactoring due to comments of PR

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 14, 2024
Minor refactoring due to comments of PR

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 14, 2024
Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 14, 2024
Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 15, 2024
Minor refactoring due to comments of PR

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 15, 2024
Minor refactoring due to comments of PR

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 15, 2024
Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 15, 2024
Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 15, 2024
Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 15, 2024
Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
akarabashov added a commit that referenced this issue Mar 15, 2024
…oot-certs

#524 Enable revocation of NOC root certificates
Abdulbois added a commit that referenced this issue Mar 19, 2024
Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 20, 2024
Fix bug related to removing certs from subject-key-id map

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 20, 2024
Fix bug related to removing certs from subject-key-id map

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 20, 2024
Fix bug related to removing certs from subject-key-id map

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Mar 20, 2024
Fix bug related to removing certs from subject-key-id map

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
akarabashov added a commit that referenced this issue Mar 20, 2024
…f-NOC-certs

#524 Enable revocation of NOC non-root certificates
@akarabashov
Copy link
Collaborator

Implemented the addition (PR: #543) and revocation (PR: #550) of NOC ROOT certificates
Unit and integration tests are added
Docs are updated.

Abdulbois added a commit that referenced this issue Apr 26, 2024
Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Apr 26, 2024
Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue Apr 27, 2024
Abdulbois added a commit that referenced this issue May 2, 2024
Abdulbois added a commit that referenced this issue May 2, 2024
Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue May 6, 2024
Abdulbois added a commit that referenced this issue May 7, 2024
Refactor impl

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
Abdulbois added a commit that referenced this issue May 7, 2024
Refactor impl

Signed-off-by: Abdulbois <[email protected]>
Signed-off-by: Abdulbois <[email protected]>
akarabashov added a commit that referenced this issue May 8, 2024
…-root-certs

#524 Enable removing NOC root certificates
@github-project-automation github-project-automation bot moved this to Done in DCL 1.4 Aug 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

No branches or pull requests

2 participants