hub
hub copied to clipboard
Implement Certificate Revocation List
This pull request includes several changes to the certificate authority's codebase, focusing on improving the handling of signing records and enhancing documentation. The most important changes include adding validation for CredentialStatus
, updating documentation to reflect the new functionality, and correcting typographical errors.
Code Enhancements:
-
certificate-authority/pb/signingRecords.go
: Added aValidate
method toCredentialStatus
to ensure all necessary fields are populated and correctly formatted. -
certificate-authority/pb/signingRecords.go
: RefactoredValidate
method inSigningRecord
to use the newCredentialStatus
validation.
Documentation Updates:
-
certificate-authority/pb/README.md
: Updated descriptions forGetSigningRecords
andDeleteSigningRecords
to reflect the new functionality of revoking certificates. Added fieldsserial
andissuer_id
to the documentation. [1] [2] [3] -
certificate-authority/pb/doc.html
: Updated HTML documentation to include new fields and corrected descriptions forGetSigningRecords
andDeleteSigningRecords
. [1] [2] [3]
Typographical Corrections:
-
certificate-authority/pb/service.proto
: Corrected typos in comments forGetSigningRecords
andDeleteSigningRecords
. [1] [2] -
certificate-authority/pb/service.swagger.json
: Corrected typos in the Swagger documentation forGetSigningRecords
andDeleteSigningRecords
. [1] [2]
Minor Changes:
-
.dockerignore
: Addedtest-local
to the ignore list. -
certificate-authority/config.yaml
: Removed bulk write configuration settings.
These changes collectively enhance the robustness of the certificate authority's functionality and improve the clarity of its documentation.