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

Feature request: Support ALPN in Deno.connectTls #11479

Closed
jaclarke opened this issue Jul 21, 2021 · 0 comments · Fixed by #12786
Closed

Feature request: Support ALPN in Deno.connectTls #11479

jaclarke opened this issue Jul 21, 2021 · 0 comments · Fixed by #12786
Labels
feat new feature (which has been agreed to/accepted) runtime Relates to code in the runtime crate

Comments

@jaclarke
Copy link

There was a previous issue to add ALPN support to the TLS api's (#6334), which was closed by this PR: #10065. However it appears that only support for ALPN in Deno.listenTls was added, is it on the roadmap to also add support for ALPN in the Deno.connectTls api as well?

Use case: We have a Deno driver for EdgeDB (https://github.com/edgedb/edgedb-deno), and the next release of EdgeDB will switch to using TLS by default (https://github.com/edgedb/rfcs/blob/master/text/1008-tls-and-alpn.rst), which requires support for ALPN in the Deno.connectTls api.

@kitsonk kitsonk added feat new feature (which has been agreed to/accepted) runtime Relates to code in the runtime crate labels Jul 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat new feature (which has been agreed to/accepted) runtime Relates to code in the runtime crate
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants