Skip to content
This repository was archived by the owner on Oct 24, 2025. It is now read-only.

Process Link header for cache requirements.#43

Merged
twifkak merged 4 commits intogoogle:mainfrom
twifkak:link
Sep 3, 2021
Merged

Process Link header for cache requirements.#43
twifkak merged 4 commits intogoogle:mainfrom
twifkak:link

Conversation

@twifkak
Copy link
Copy Markdown
Contributor

@twifkak twifkak commented Sep 2, 2021

Parse and transform the Link header to meet the Google SXG Cache requirements
at https://github.com/google/webpackager/blob/main/docs/cache_requirements.md.

Directives are omitted if they have an invalid rel or crossorigin or an unknown
param name. Hrefs are converted to absolute URLs. Preloads are limited to 20.
imagesrcset and imagesizes are not yet validated.

A future commit will compute header-integrity and add appropriate
allowed-alt-sxg directives.

Addresses #13.

Parse and transform the Link header to meet the Google SXG Cache requirements
at https://github.com/google/webpackager/blob/main/docs/cache_requirements.md.

Directives are omitted if they have an invalid rel or crossorigin or an unknown
param name. Hrefs are converted to absolute URLs. Preloads are limited to 20.
imagesrcset and imagesizes are not yet validated.

A future commit will compute header-integrity and add appropriate
allowed-alt-sxg directives.
@twifkak twifkak requested a review from antiphoton September 2, 2021 23:06
twifkak and others added 3 commits September 2, 2021 17:58
Co-authored-by: Boxiao Cao <9083193+antiphoton@users.noreply.github.com>
@twifkak twifkak merged commit 8b4abe4 into google:main Sep 3, 2021
@twifkak twifkak deleted the link branch September 3, 2021 01:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants