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

Add TLS 1.3 to Go and Caddy #181

Closed
wants to merge 1 commit into from
Closed

Add TLS 1.3 to Go and Caddy #181

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Dec 6, 2018

This PR adds support for TLS 1.3 (RFC 8446) to Go and Caddy. TLS 1.3 support was recently added to Go. Caddy is written in Go and if you compile it with the latest branch of Go and apply a tiny patch to it, Caddy also supports TLS 1.3. To see an example of a site using Caddy with TLS 1.3, click here.

@igrigorik
Copy link
Owner

igrigorik commented Dec 6, 2018

Hmm, what's blocking the patch from being merge into mainline? I'd love to see that be the default before we flip the bit.

p.s. thanks for drafting the update!

@ghost
Copy link
Author

ghost commented Dec 6, 2018

@igrigorik The developer of Caddy (@mholt) wants to wait for the next point release of Go (1.12) before merging it into mainline since compiling Caddy with TLS 1.3 support requires you to use a compiled version of Go's master branch instead of its current point release (1.11.2). To my knowledge, Go 1.12 is scheduled to be released in Feb 2019.

Edit: I accidentally a word.

@igrigorik
Copy link
Owner

Gotcha, thanks for the helpful context. Given that this still requires build-from-master and patches, I'd propose we wait until this work makes it into stable?

@ghost
Copy link
Author

ghost commented Dec 6, 2018

@igrigorik Sure thing. I'll update this pull request once TLS 1.3 support lands in the stable versions of Caddy and Go.

@mholt
Copy link

mholt commented Dec 6, 2018

Thanks for the ping, and for this patch. It's a little early though 😄 All of this is correct. Shortly after Go 1.12 is released in February, Caddy will default to TLS 1.3, and we'll be tagging Caddy v1.0 sometime around then too.

@ghost
Copy link
Author

ghost commented Feb 26, 2019

@igrigorik @mholt TLS 1.3 support has landed in the stable versions of Go and Caddy. Can you merge this PR now?

@igrigorik
Copy link
Owner

Yep! One more thing before I merge...

Now that it's live in stable, do we have doc URLs we can point to, instead of GitHub issues?

@mholt
Copy link

mholt commented Mar 3, 2019

By Monday we should. I will update this issue then.

Copy link

@mholt mholt left a comment

Choose a reason for hiding this comment

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

I've suggested relevant links!

index.html Show resolved Hide resolved
index.html Show resolved Hide resolved
@igrigorik igrigorik mentioned this pull request Mar 6, 2019
@midzer
Copy link
Contributor

midzer commented Mar 6, 2019

Looks like OPs branch is gone: unknown repository

@mholt proposed changes look good to me

@mholt
Copy link

mholt commented Mar 6, 2019

@WhaleHub Do you want to finish this up? (Where'd the repo go?)

@ghost ghost closed this Mar 10, 2019
@ghost ghost mentioned this pull request Mar 10, 2019
@ghost
Copy link
Author

ghost commented Mar 10, 2019

@mholt @midzer @igrigorik I opened a new PR with the links suggested by mholt: #192

This pull request was closed.
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.

3 participants