feat: custom pushAssets function to control http2 push headers

[awronski]

Types of changes

• Bug fix (a non-breaking change which fixes an issue)
• New feature (a non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Description

Resolves: #4011

Adds pushAssets(req, res, publicPath, preloadFiles) option to http2.
Function returns array of links. Now we can conditionally decide which assets we want to push.

Undocumented filter function shouldPush was marked as deprecated

Checklist:

• My change requires a change to the documentation.
• I have updated the documentation accordingly. (PR: #)
• I have added tests to cover my changes (if not applicable, please state why)
• All new and existing tests are passing.

[pi0]

LGTM. As this is almost a refactor i think we can add tests later.

[awronski]

@pi0 I am thinking about removing publicPath from arguments of pushAssets(..). Because user has it in nuxt.config.js. What do you think?

[pi0]

@awronski This value may be different. It should be read from client manifest to follow the value passed at compile time.

[clarkdo]

Doc and release note need to be updated

[awronski]

@clarkdo I will update docs

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.