Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add support for user TLS authentication #7016 #7035

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
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dsugar100
Copy link

This allows for the use case where a user is providing an openssl key and certificate as their authentication to an HTTPS server.

New option GIT_OPT_SET_SSL_USER_KEY to set paths for user private key and certificate files.
New option GIT_OPT_SET_SSL_PASSWORD_CALLBACK provides a means for a callback function to prompt for password to unlock the key.

This allows for the use case where a user is providing an openssl
key and certificate as their authentication to an HTTPS server.

New option GIT_OPT_SET_SSL_USER_KEY to set paths for user private
key and certificate files.
New option GIT_OPT_SET_SSL_PASSWORD_CALLBACK provides a means for
a callback function to prompt for password to unlock the key.
@ethomson
Copy link
Member

Thanks for the PR; I've been noodling on this a little bit, and I wonder if a library option is the best place for this.

In particular, I wonder if it's really server / remote specific. Is there a situation where you would want to present two different certificates to two different remotes? Or - perhaps more likely - you would want to present user / client certificates to an internal server, but you would want to present no client certs to a public server (like github.com?)

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.

2 participants