Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Do not render CRLs in memory #1373

Merged
merged 1 commit into from
May 3, 2023
Merged

Do not render CRLs in memory #1373

merged 1 commit into from
May 3, 2023

Conversation

azazeal
Copy link
Contributor

@azazeal azazeal commented May 3, 2023 •
edited

This PR refactors the way PEM-encoded CRLs are served so that they're directly rendered into the underlying http.ResponseWriter.

@github-actions github-actions bot added the needs triage Waiting for discussion / prioritization by team label May 3, 2023
@azazeal azazeal requested a review from maraino May 3, 2023 17:24
@azazeal azazeal marked this pull request as ready for review May 3, 2023 17:24
@azazeal azazeal requested a review from hslatman May 3, 2023 17:24
@azazeal azazeal removed the needs triage Waiting for discussion / prioritization by team label May 3, 2023
@azazeal azazeal merged commit d797941 into master May 3, 2023
15 checks passed
@azazeal azazeal deleted the crlflush branch May 3, 2023 20:49
@hslatman hslatman added this to the v0.24.2 milestone May 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hslatman hslatman hslatman approved these changes

@maraino maraino Awaiting requested review from maraino

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v0.24.2
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@azazeal @hslatman