Skip to content
This repository was archived by the owner on Nov 1, 2018. It is now read-only.

[AspNetCoreModule] Use HttpResponse.PushPromise #160

Closed
benaadams opened this issue May 1, 2016 · 3 comments
Closed

[AspNetCoreModule] Use HttpResponse.PushPromise #160

benaadams opened this issue May 1, 2016 · 3 comments

Comments

@benaadams
Copy link

benaadams commented May 1, 2016

To more fully utilize http/2 where available, if response includes Link header with a local path and rel of preload add PushPromise for that path.

This will enable the reverse-proxy (IIS) to "value add" more http/2 features to Kestrel when used in that configuration. (e.g. Win10, Server 2016, nano server)

e.g.

Link: /css/site.css; rel=preload;

Add

HttpResponse.PushPromise("/css/site.css");

Or

Link:  /css/site.css; rel=preload; as=style;,/fonts/font1.woff; rel=preload; as=font;,/fonts/font2.woff; rel=preload; as=font;

Add

HttpResponse.PushPromise("/css/site.css");
HttpResponse.PushPromise("/fonts/font1.woff");
HttpResponse.PushPromise("/fonts/font2.woff");

IIS PushPromise as per http://blogs.iis.net/davidso/http2

Link header parsing to add push promises as per https://blog.cloudflare.com/announcing-support-for-http-2-server-push-2/

/cc @DamianEdwards

@Tratcher Tratcher added this to the Backlog milestone May 1, 2016
@d0pare
Copy link

d0pare commented May 1, 2016

Really must have feature.

Thanks @benaadams

@Tratcher
Copy link
Member

RE: aspnet/HttpAbstractions#371

@Tratcher
Copy link
Member

This issue was moved to aspnet/AspNetCoreModule#28

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants