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.

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

[Feature]: Load client certificates from RAM #32200

Closed
BenMcLean opened this issue Aug 16, 2024 · 0 comments · Fixed by #32210
Closed

[Feature]: Load client certificates from RAM #32200

BenMcLean opened this issue Aug 16, 2024 · 0 comments · Fixed by #32210
Assignees
Labels

Comments

@BenMcLean
Copy link

BenMcLean commented Aug 16, 2024

🚀 Feature Request

The API for client certificates should accept the actual certificate, not a path to a file containing the certificate.

Example

The certificate could be coming in at runtime from a physical hardware token or from some other source that isn't a file.

Motivation

The current implementation of the new client certificates feature from #1799 and #32004 has the API expecting a file path to receive the client certificates. However, the certificates might not be coming from a file: they could instead get generated at runtime and only exist in RAM. It would be preferable and also possibly more secure for the API to accept the actual certificate instead of a path to a file containing the certificate because a file might not strictly need to be written to do this operation. The certificates could come from a file or they could come from anything as long as the certificates are provided somehow.

I discussed this request with @mxschmitt on Discord.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants