Skip to content

Conversation

@aarongable
Copy link
Contributor

@aarongable aarongable commented Oct 27, 2025

The IncludeCRLDistributionPoints config item is ignored -- we include a CRLDP no matter what. Therefore the stanza requiring that one be set in the config is redundant and useless, and prevents us from cleaning up that config item.

It is likely that, if and when we address #7673 to drop the CRLDP from short-lived certs, we will use a new "OmitRevocation" config field.

The IncludeCRLDistributionPoints config item is ignored -- we include a CRLDP no matter what. Therefore the stanza requiring that one be set in the config is redundant and useless, and prevents us from cleaning up that config item.

It is likely that, if and when we are able to drop the CRLDP from short-lived certs, we will use a new "OmitRevocation" config field.
@aarongable aarongable marked this pull request as ready for review October 27, 2025 18:26
@aarongable aarongable requested a review from a team as a code owner October 27, 2025 18:26
Copy link
Contributor

@jsha jsha left a comment

Choose a reason for hiding this comment

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

Looks good. PR description should mention #7673, which is mentioned in a TODO here.

@aarongable aarongable merged commit 529776e into main Oct 27, 2025
12 checks passed
@aarongable aarongable deleted the aarongable-patch-1 branch October 27, 2025 18:43
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.

4 participants