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

Making tls-commit and tls-ports mutually exclusive #207

Closed
wants to merge 2 commits into from

Conversation

martinthomson
Copy link
Contributor

This probably isn't the end of this issue though. We require that an opportunistic upgrade uses the same hostname. The text I added here lifts the requirement regarding port numbers, but should it also say that the hostname needs to remain the same?

I've left this open here, but I think that we probably should leave the hostname requirement in place. That way, we still retain some of the aspects of http:// authority.

@MikeBishop
Copy link
Contributor

I disagree. As soon as you're fully authenticating the TLS certificate, you're doing vanilla Alt-Svc and this draft could be totally out of the picture. The only thing you're gaining from the http-opportunistic resource is a commitment that the TLS Alt-Svc will remain available and a suggestion that the client refuse to fall back to the plain-text origin. I don't see that same-host plays into that.

Alternately phrased -- what about the alternative being on a different host implies the origin should not be able to make a commitment that it will remain available?

@martinthomson
Copy link
Contributor Author

Sorry about being so slow, but @MikeBishop, if I understand what you are implying, you are suggesting that we should remove the host restriction if the certificate fits. I was thinking about a client using the process we outline in the draft to arrive at such a host, which isn't really possible.

However, I guess we have to accept that a client might learn of this resource from an HTTPS connection to some other host (maybe because of connection coalescing). In which case, then I find that I agree. Some more careful words needed, I guess.

@martinthomson
Copy link
Contributor Author

I think that this has been OBE.

@martinthomson martinthomson deleted the commit_ports branch September 28, 2016 06:58
@mnot
Copy link
Member

mnot commented Sep 28, 2016

Ah, sorry - forgot to look at the pulls.

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

Successfully merging this pull request may close these issues.

None yet

3 participants