Skip to content

TLS ALPN support #6334

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

Closed
yearsyan opened this issue Jun 17, 2020 · 4 comments · Fixed by #10065
Closed

TLS ALPN support #6334

yearsyan opened this issue Jun 17, 2020 · 4 comments · Fixed by #10065
Assignees
Labels
cli related to cli/ dir feat new feature (which has been agreed to/accepted) tls Issues related to TLS implementation

Comments

@yearsyan
Copy link

yearsyan commented Jun 17, 2020

Feature request

I thiink it is necessary to support alnp. For example, if a developer want to create a http2 server, alpn can make handshake quicker.

@kitsonk
Copy link
Contributor

kitsonk commented Jun 17, 2020

ALPN would need to be implemented as part of HTTP/2, so effectively duplicate of #3995.

@lucacasonato
Copy link
Member

@kitsonk Not per se - if the HTTP/2 server is Rust based, then an implementation of ALPN for Deno.connectTls or Deno.listenTls would need to happen separately from HTTP/2 right?

@kitsonk
Copy link
Contributor

kitsonk commented Jun 17, 2020

I mean we wouldn't have to implemented ALPN to implement HTTP/2, but it doesn't make sense... the whole thing should be addressed as one (likely along with moving HTTP/1.1 into Rust as well which is something we said we were going to have to do).

@yearsyan
Copy link
Author

@kitsonk I think an implementation of ALPN for Deno.connectTls or Deno.listenTls is good for developer who want to create a protocol over tls, HTTP/2 is not the only protocol needs ALPN.

@bartlomieju bartlomieju added cli related to cli/ dir suggestion suggestions for new features (yet to be agreed) labels Jul 14, 2020
@bartlomieju bartlomieju added the tls Issues related to TLS implementation label Oct 14, 2020
@lucacasonato lucacasonato added feat new feature (which has been agreed to/accepted) and removed suggestion suggestions for new features (yet to be agreed) labels Apr 7, 2021
@lucacasonato lucacasonato self-assigned this Apr 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cli related to cli/ dir feat new feature (which has been agreed to/accepted) tls Issues related to TLS implementation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants