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

RFC 9266: Channel Bindings for TLS 1.3 support #392

Open
Neustradamus opened this issue Aug 1, 2022 · 5 comments
Open

RFC 9266: Channel Bindings for TLS 1.3 support #392

Neustradamus opened this issue Aug 1, 2022 · 5 comments

Comments

@Neustradamus
Copy link

Neustradamus commented Aug 1, 2022

Can you add the support of RFC 9266: Channel Bindings for TLS 1.3?

Little details, to know easily:

  • tls-unique for TLS =< 1.2
  • tls-server-end-point
  • tls-exporter for TLS = 1.3

Thanks in advance.

Linked to:

@johnweldon
Copy link
Member

Thank you for submitting this request - you're welcome to propose a PR.

@cpuschma
Copy link
Member

cpuschma commented Aug 1, 2022

It looks like you're on a campaign to ask maintainers to add this as a new feature in several libs, including the standard libs provided by Go: golang/go#54103

I think it would be best to wait for the standard libraries to implement this and make it available to the devs, than to implement this complex and important part ourselves. SSL/TLS are in OSI layer 6 anyway, LDAP (and our implementation) in layer 7.

Personally, I don't see it as our responsibility to provide this part (TLS 1.3 Channel Binding) either. The tasks of the library should be clearly coordinated. As an example: I don't see the task of connection pooling in our responsibility, because there is no clear consensus on how to do it properly.

@Neustradamus
Copy link
Author

Dear @go-ldap team, @johnweldon, @cpuschma,

In first, I wish you a Happy New Year 2024!

Have you progressed on the support?

Maybe you can look here:

@Neustradamus
Copy link
Author

@cpuschma: Excellent, where is the commit fix?

If there is not, it has been not solved, you need to reopen this ticket.

@cpuschma
Copy link
Member

I've been in the wrong issue, sorry 'bout that.

@cpuschma cpuschma reopened this Apr 30, 2024
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

No branches or pull requests

3 participants