Skip to content

Comments

Remove root ca terminology#66

Closed
HBrock wants to merge 4 commits intomainfrom
remove_root-CA_terminology
Closed

Remove root ca terminology#66
HBrock wants to merge 4 commits intomainfrom
remove_root-CA_terminology

Conversation

@HBrock
Copy link
Contributor

@HBrock HBrock commented Jan 16, 2025

Implements #65

Copy link
Collaborator

@johngray-dev johngray-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a few minor comments. Generally it seems to make things clearer I think, but it does differ in language from other RFCs

DN, e.g., indicating a generation identifier like the year of issuance or
a version number, for instance in an OU element. How to bridge trust to
the new root CA certificate in a CA DN change or a cross-certificate scenario
a new CA certificate in a CA DN change or a cross-certificate scenario
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could also say "How to bridge trust to a newly trusted CA Certificate in a CA DN change..."

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

Profile of how a Certificate structure may be "self-signed". These
structures are used for distribution of CA public keys. This can
Profile of how a certificate structure may be "self-signed". These
structures are used for distribution of new trusted CA public keys as self-signed certificate. This can
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I would say "newly trusted"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

-- in later PKI messages (for encryption)
RootCaKeyUpdate optionally present, with
relevant value
RootCaKeyUpdate optionally present, with relevant value
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this name change from RootCAKeyUpdate to TrustedCaKeyUpdate?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be great to also update the types. But this would be a breaking change for RFC 9483 because we introduced the new types already with RFC 9480 and we also registered some OIDs using the old language. Therefore, I think we should keep the syntax and address this with the note.

@HBrock
Copy link
Contributor Author

HBrock commented Jan 27, 2025

Option 1 was rejected in favor of option 2, #67 was merged

@HBrock HBrock closed this Jan 27, 2025
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.

2 participants