Skip to content

Client certificate authentication support for binary caches #13002

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

Open
2 tasks done
vlaci opened this issue Apr 11, 2025 · 1 comment · May be fixed by #13030
Open
2 tasks done

Client certificate authentication support for binary caches #13002

vlaci opened this issue Apr 11, 2025 · 1 comment · May be fixed by #13030
Labels
feature Feature request or proposal

Comments

@vlaci
Copy link

vlaci commented Apr 11, 2025

Is your feature request related to a problem?

In our setup, we use client certificate authentication extensively. It would be very easy for us to deploy a binary cache and setup authentication the same way.

Proposed solution

  • Extend nix.conf with settings for client certificate and private keys.
    It can be (1) global configuration with a single keypair, or (2) per substituter URL.

  • modify FileTransfer to configure the requierd key/cert pair in the curl request

If my limited understanding of the codebase is correct, this would also make fetchers work with certificates, which would be a nice bonus but not required for us.

If the feature request is acceptable in principle, I am willing to implement and test it.

Alternative solutions

Alternatively, we could setup basic authentication, but it would be more painful from identity management perspective.

Additional context

#690 which initially proposed client certificate authentication as a possible authentication mechanism.

Checklist


Add 👍 to issues you find important.

@vlaci vlaci added the feature Feature request or proposal label Apr 11, 2025
@Mic92 Mic92 added this to Nix team Apr 12, 2025
@github-project-automation github-project-automation bot moved this to To triage in Nix team Apr 12, 2025
@Mic92
Copy link
Member

Mic92 commented Apr 12, 2025

Mtls doesn't sound like a bad idea. I bring this up for discussion in the next meeting.

@vlaci vlaci linked a pull request Apr 15, 2025 that will close this issue
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Feature request or proposal
Projects
Status: To triage
Development

Successfully merging a pull request may close this issue.

2 participants