tomschlick / laravel-http2-server-push

A middleware package for Laravel to enable server push for your script, style, and image assets.
MIT License
167 stars 20 forks source link

Add helpers for other types of content #11

Open kohenkatz opened 8 years ago

kohenkatz commented 8 years ago

I can do a PR for this, but first I wanted to gauge interest.

The preload spec that defines server push lists the valid "request destination" types for the as attribute of the Link header. Of those, only image, script, and style have helpers in this package. While those are likely the most commonly used, I think that we should make it easier for users to specify the other types.

There are two ways that I can think of to do this:

  1. Add a generic helper pushFile that has a second argument which defines the type.
  2. Add helpers for all of the types.

Here is the example list of types from the documentation (including the three types already supported):

The spec also defines the following types, but they are not listed in this part of the documentation:

Thoughts?

tomschlick commented 8 years ago

Yeah this has been on my todo list since I created the package but I haven't found the time yet...

I'd be all for a PR if you have the time to do so!

Maybe we go for a generic pushItem($path, $type) method but keep the ones for image, style, script.

kohenkatz commented 8 years ago

Some time in the next few days I will try to get a PR in for a pushItem helper.

In addition to $path, and $type, there will also be a $mimeType argument for things like font to use. It will default to null because it isn't needed everywhere.

tomschlick commented 8 years ago

Sounds good. Thanks!

regiszanandrea commented 6 years ago

Hi everyone,

I created a PR that supports Fonts, could be useful for you

PR: https://github.com/tomschlick/laravel-http2-server-push/pull/19