Masatoshi Kimura [:emk] Reporter
	Description • 9 years ago

Will HTTP-WG keep messing around with the draft until our ClientHello header is filled up with the ALPN extension?

h2-14 was left because it was an implementation draft. But are h2-15 and h2-16 need to be advertised?

	Patrick McManus [:mcmanus]
	Comment 1 • 9 years ago

I expect -17 to go to the rfc editor, which means it will really be final as h2 and no-one will try and negotiate -17 in particular instead of -17. So we can probably wontfix this bug, but I'll wait for the standards action to happen before closing this.

We can kill all the draft versions soonish. gecko 40 sounds about right.

fwiw they've all been used at various points with overlap at the same point between all 3 versions. "implementation draft" isn't really a thing in practice and it really doesn't make a lot of sense if you want to get real feedback on the changes.

	Patrick McManus [:mcmanus]
	Comment 2 • 9 years ago

based on -17 going to proposed standard, we can change this bug to be remove draft-standard support starting in gecko 40. i.e. h2 only. (this should not impact spdy - that's a different bug and a different timeline.)

	u408661 Assignee
	Comment 3 • 9 years ago

I've been looking forward to this ever since the day I started adding all those alpn tokens :) I'll write the patch, and we can have it hanging around waiting for 40 to be on m-c.

	u408661 Assignee
	Comment 5 • 9 years ago

Huh, git-bz fail - it put my comment into the commit message of the patch instead of the comment (maybe I put it in the wrong place in the file, who knows). Anyway:

This should hit all the high points. Obviously, I'll wait until m-c becomes gecko 40 to land (assuming this is r+'d)

https://treeherder.mozilla.org/#/jobs?repo=try&revision=e119e99603ef

	Patrick McManus [:mcmanus]
	Comment 6 • 9 years ago

Comment on attachment 8573426 [details] [diff] [review]
patch

Review of attachment 8573426 [details] [diff] [review]:
-----------------------------------------------------------------

landing is contingent on having a RFC number to point to

	u408661 Assignee
	Comment 7 • 9 years ago

Added RFC numbers in appropriate files. Hooray!

remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/dc3c55cedb3f

	Phil Ringnalda (:philor)
	Comment 8 • 9 years ago

https://hg.mozilla.org/mozilla-central/rev/dc3c55cedb3f

	Kohei Yoshino [:kohei]
	Comment 9 • 9 years ago

Added the site compatibility doc: https://www.fxsitecompat.com/en-US/docs/2015/http-2-draft-is-no-longer-supported/

	Jean-Yves Perrier [:teoli]
	Comment 10 • 9 years ago

Added a note: https://developer.mozilla.org/en-US/Firefox/Releases/41#Networking