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

TODO: add TLS channel binding #13483

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bk2204
Copy link

@bk2204 bk2204 commented Apr 25, 2024

TLS channel binding is a security improvement that cryptographically ties authentication to a specific TLS connection, preventing TLS MITM proxies from successfully authenticating. This would be valuable for many security-sensitive users, so add it to the TODO list.

Copy link
Member

@danielgustafsson danielgustafsson left a comment

Choose a reason for hiding this comment

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

No objection to adding it to the TODO

TLS channel binding is a security improvement that cryptographically
ties authentication to a specific TLS connection, preventing TLS MITM
proxies from successfully authenticating.  This would be valuable for
many security-sensitive users, so add it to the TODO list.
@bk2204
Copy link
Author

bk2204 commented May 4, 2024

I noticed this had grown a conflict, so I rebased onto the latest version of the main branch and renumbered the new entry.

@christophvw
Copy link

It is already supported with SCHANNEL backend.

and a pull request for openssl backend is available as well:
#13098

@bk2204
Copy link
Author

bk2204 commented May 14, 2024

This proposal is not specific to SPNEGO. I'm proposing an approach where a user can use SPNEGO or a custom authentication scheme via a callback.

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

Successfully merging this pull request may close these issues.

None yet

3 participants