Skip to content

Commit

Permalink
Clarified the intent of the uri field in the certificate authority (#140
Browse files Browse the repository at this point in the history
)

* Clarified the intent of the uri field in the certificate authority

Signed-off-by: Fredrik Skogman <[email protected]>

* Fixed spelling

Signed-off-by: Fredrik Skogman <[email protected]>

* Generate files based on the updated protobuf files.

Signed-off-by: Fredrik Skogman <[email protected]>

---------

Signed-off-by: Fredrik Skogman <[email protected]>
  • Loading branch information
kommendorkapten authored Sep 18, 2023
1 parent c8b0b0f commit d382fc3
Show file tree
Hide file tree
Showing 10 changed files with 32 additions and 10 deletions.
2 changes: 1 addition & 1 deletion gen/jsonschema/schemas/CertificateAuthority.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
},
"uri": {
"type": "string",
"description": "The URI at which the CA can be accessed."
"description": "The URI identifies the certificate authority. It is RECOMMENDED that the URI is the base URL for the certificate authority, that can be provided to any SDK/client provided by the certificate authority to interact with the certificate authority."
},
"certChain": {
"$ref": "#/definitions/dev.sigstore.common.v1.X509CertificateChain",
Expand Down
2 changes: 1 addition & 1 deletion gen/jsonschema/schemas/Input.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -516,7 +516,7 @@
},
"uri": {
"type": "string",
"description": "The URI at which the CA can be accessed."
"description": "The URI identifies the certificate authority. It is RECOMMENDED that the URI is the base URL for the certificate authority, that can be provided to any SDK/client provided by the certificate authority to interact with the certificate authority."
},
"certChain": {
"$ref": "#/definitions/dev.sigstore.common.v1.X509CertificateChain",
Expand Down
2 changes: 1 addition & 1 deletion gen/jsonschema/schemas/TrustedRoot.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -179,7 +179,7 @@
},
"uri": {
"type": "string",
"description": "The URI at which the CA can be accessed."
"description": "The URI identifies the certificate authority. It is RECOMMENDED that the URI is the base URL for the certificate authority, that can be provided to any SDK/client provided by the certificate authority to interact with the certificate authority."
},
"certChain": {
"$ref": "#/definitions/dev.sigstore.common.v1.X509CertificateChain",
Expand Down
7 changes: 6 additions & 1 deletion gen/pb-go/trustroot/v1/sigstore_trustroot.pb.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion gen/pb-rust/schemas/CertificateAuthority.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
},
"uri": {
"type": "string",
"description": "The URI at which the CA can be accessed."
"description": "The URI identifies the certificate authority. It is RECOMMENDED that the URI is the base URL for the certificate authority, that can be provided to any SDK/client provided by the certificate authority to interact with the certificate authority."
},
"certChain": {
"$ref": "#/definitions/dev.sigstore.common.v1.X509CertificateChain",
Expand Down
2 changes: 1 addition & 1 deletion gen/pb-rust/schemas/Input.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -516,7 +516,7 @@
},
"uri": {
"type": "string",
"description": "The URI at which the CA can be accessed."
"description": "The URI identifies the certificate authority. It is RECOMMENDED that the URI is the base URL for the certificate authority, that can be provided to any SDK/client provided by the certificate authority to interact with the certificate authority."
},
"certChain": {
"$ref": "#/definitions/dev.sigstore.common.v1.X509CertificateChain",
Expand Down
2 changes: 1 addition & 1 deletion gen/pb-rust/schemas/TrustedRoot.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -179,7 +179,7 @@
},
"uri": {
"type": "string",
"description": "The URI at which the CA can be accessed."
"description": "The URI identifies the certificate authority. It is RECOMMENDED that the URI is the base URL for the certificate authority, that can be provided to any SDK/client provided by the certificate authority to interact with the certificate authority."
},
"certChain": {
"$ref": "#/definitions/dev.sigstore.common.v1.X509CertificateChain",
Expand Down
9 changes: 8 additions & 1 deletion gen/pb-typescript/src/__generated__/sigstore_trustroot.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

7 changes: 6 additions & 1 deletion protos/sigstore_trustroot.proto
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,12 @@ message CertificateAuthority {
// The root certificate MUST be self-signed, and so the subject and
// issuer are the same.
dev.sigstore.common.v1.DistinguishedName subject = 1;
// The URI at which the CA can be accessed.
// The URI identifies the certificate authority.
//
// It is RECOMMENDED that the URI is the base URL for the certificate
// authority, that can be provided to any SDK/client provided
// by the certificate authority to interact with the certificate
// authority.
string uri = 2;
// The certificate chain for this CA.
dev.sigstore.common.v1.X509CertificateChain cert_chain = 3;
Expand Down

0 comments on commit d382fc3

Please sign in to comment.