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

HTTP/2 #731

Closed
XhmikosR opened this issue Aug 5, 2016 · 16 comments
Closed

HTTP/2 #731

XhmikosR opened this issue Aug 5, 2016 · 16 comments
Labels

Comments

@XhmikosR
Copy link
Collaborator

XhmikosR commented Aug 5, 2016

Would be nice if we served with HTTP/2.

@jdorfman
Copy link
Member

jdorfman commented Aug 5, 2016

Back in the day, when I tested with SPDY on, I found single files like bootstrap.min.js actually hurt PLT rather than improve. I wish I saved the test results. With that said let's do some tests and see if my measuring was off. It could make a great blog post as well.

@jdorfman
Copy link
Member

@jmervine when will Heroku have H/2?

@jmervine
Copy link
Contributor

Unfortunately, I cannot comment on our roadmap. HTTP/2 is certainly something Heroku is interested in supporting in the future.

@XhmikosR
Copy link
Collaborator Author

Any updates on this? The CDN files are already using HTTP/2; only the website which is on Heroku doesn't use HTTP/2.

@XhmikosR XhmikosR removed the cdn label Dec 22, 2016
@jdorfman
Copy link
Member

Still not supported: https://devcenter.heroku.com/articles/http-routing#not-supported

When it is supported, it will be on the front page of HN and we will enable it. For now, closing

@jdorfman jdorfman added cdn and removed www labels Nov 30, 2017
@jdorfman
Copy link
Member

Reopening this issue after receiving an email from a user. Going forward we will only be talking about H/2 for cdn (not www).

@jdorfman jdorfman reopened this Nov 30, 2017
@XhmikosR
Copy link
Collaborator Author

The CDN files are already being served with http/2.

@jdorfman
Copy link
Member

I thought they were but they aren't.

image

Also checked with https://h2.nix-admin.com/#

@XhmikosR
Copy link
Collaborator Author

True, I also thought they were.

@jdorfman
Copy link
Member

Monday we will enable it and watch the logs.

@petarpetrovic
Copy link

petarpetrovic commented Nov 30, 2017

CDN assets are definitely not being served over HTTP/2 right now:

Petars-MacBook-Pro:~ ppetrovic$ curl -I https://maxcdn.bootstrapcdn.com/bootstrap/3.3.7/css/bootstrap.min.css --http2
HTTP/1.1 200 OK
Date: Thu, 30 Nov 2017 21:46:33 GMT
Content-Type: text/css
Content-Length: 121200
Connection: keep-alive
Last-Modified: Mon, 25 Jul 2016 16:08:01 GMT
ETag: "ec3bb52a00e176a7181d454dffaea219"
Server: NetDNA-cache/2.2
Expires: Sun, 25 Nov 2018 21:46:33 GMT
Cache-Control: max-age=31104000
Vary: Accept-Encoding
Access-Control-Allow-Origin: *
X-Hello-Human: Say hello back! @getBootstrapCDN on Twitter
X-Cache: HIT
Accept-Ranges: bytes

That will change on Monday 😎

@XhmikosR
Copy link
Collaborator Author

Maybe they were in the past? Because I'd swear they were...

@petarpetrovic
Copy link

It's possible HTTP/2 never got reactivated during the migration from the legacy MaxCDN infrastructure to StackPath. Unfortunately, I don't have sufficient information to confirm that, it's just my educated guess. Either way, it's getting reactivated soon.

@XhmikosR
Copy link
Collaborator Author

Yup, I'm right.

See https://gtmetrix.com/compare/CPIKVUkX/HiaFl5nH#waterfall

@jdorfman
Copy link
Member

jdorfman commented Dec 4, 2017

Checklist:

@jdorfman
Copy link
Member

jdorfman commented Dec 4, 2017

I always monitor Twitter so this case is closed.

@jdorfman jdorfman closed this as completed Dec 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants