Skip to content

Conversation

@maen-bn
Copy link
Contributor

@maen-bn maen-bn commented Jan 21, 2026

Fixes #8341
IN-8593 tracked the corresponding prod config change

@maen-bn maen-bn requested a review from a team as a code owner January 21, 2026 16:14
aarongable
aarongable previously approved these changes Jan 21, 2026
Comment on lines 36 to 38
// ServeRenewalInfo exposes the renewalInfo endpoint in the directory and for
// GET requests. WARNING: This feature is a draft and highly unstable.
ServeRenewalInfo bool
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change

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.

Generally looks good other than that one fix. Thanks!

@maen-bn
Copy link
Contributor Author

maen-bn commented Jan 27, 2026

Generally looks good other than that one fix. Thanks!

Great. Apologies about that. Was a bad merge on my part

@aarongable aarongable requested a review from jsha January 27, 2026 17:52
@aarongable
Copy link
Contributor

Thanks for this contribution! For future reference, no need to merge main at the very end, unless an actual merge conflict has shown up.

@aarongable aarongable merged commit 1afad15 into letsencrypt:main Jan 27, 2026
29 of 31 checks passed
@aarongable aarongable changed the title Remove ServeRenewalInfo feature flag Deprecate ServeRenewalInfo feature flag Jan 27, 2026
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.

Remove ServeRenewalInfo feature flag

3 participants