Skip to content

Commit

Permalink
Add reason to certificate status
Browse files Browse the repository at this point in the history
This commit adds a reason string to the certificate status. It breaks
backward compatibility, but this is ok because this method is not
currently used.
  • Loading branch information
maraino committed Jun 13, 2024
1 parent 2e5816a commit b9749eb
Show file tree
Hide file tree
Showing 2 changed files with 49 additions and 28 deletions.
67 changes: 43 additions & 24 deletions majordomo.pb.go

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

10 changes: 6 additions & 4 deletions spec/linkedca/majordomo.proto
Original file line number Diff line number Diff line change
Expand Up @@ -251,8 +251,9 @@ message GetCertificateStatusRequest {

message GetCertificateStatusResponse {
RevocationStatus status = 1;
RevocationReasonCode reason_code = 2;
google.protobuf.Timestamp revoked_at = 3;
string reason = 2;
RevocationReasonCode reason_code = 3;
google.protobuf.Timestamp revoked_at = 4;
}

message GetSSHCertificateStatusRequest {
Expand All @@ -261,6 +262,7 @@ message GetSSHCertificateStatusRequest {

message GetSSHCertificateStatusResponse {
RevocationStatus status = 1;
RevocationReasonCode reason_code = 2;
google.protobuf.Timestamp revoked_at = 3;
string reason = 2;
RevocationReasonCode reason_code = 3;
google.protobuf.Timestamp revoked_at = 4;
}

0 comments on commit b9749eb

Please sign in to comment.