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

[compression-dictionary] Added link relation #2783

Merged
merged 2 commits into from
May 6, 2024
Merged

Conversation

pmeenan
Copy link
Contributor

@pmeenan pmeenan commented Apr 30, 2024

This adds the definition for the compression-dictionary link relation type and relevant IANA registration information.

The link relation provides a way to trigger the fetch of a stand-alone dictionary (e.g. for compressing HTML). The actual dictionary processing follows the standard path and headers once the fetch has been initiated.

There is discussion in issue #2772 as well as a related issue in the HTML spec.

In Chrome's current origin trial and the initial explainers this is the dictionary relation type but that term is too generic for this specific case.

Fix #2772

Copy link
Contributor

@yoavweiss yoavweiss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM % nits

draft-ietf-httpbis-compression-dictionary.md Outdated Show resolved Hide resolved
draft-ietf-httpbis-compression-dictionary.md Outdated Show resolved Hide resolved
draft-ietf-httpbis-compression-dictionary.md Outdated Show resolved Hide resolved
draft-ietf-httpbis-compression-dictionary.md Outdated Show resolved Hide resolved
draft-ietf-httpbis-compression-dictionary.md Outdated Show resolved Hide resolved
draft-ietf-httpbis-compression-dictionary.md Outdated Show resolved Hide resolved
draft-ietf-httpbis-compression-dictionary.md Outdated Show resolved Hide resolved
draft-ietf-httpbis-compression-dictionary.md Outdated Show resolved Hide resolved
@pmeenan
Copy link
Contributor Author

pmeenan commented May 1, 2024

Thanks. Flows better with the suggested changes (PR updated).

@pmeenan pmeenan merged commit 355d23b into httpwg:main May 6, 2024
1 check passed
@pmeenan pmeenan deleted the relation branch May 8, 2024 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

Add link relation to trigger compression dictionary fetches
3 participants