-
Notifications
You must be signed in to change notification settings - Fork 16
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 or inheritence for headers #116
Comments
toomim
changed the title
Add compression or inheritence for headers
Compression or inheritence for headers
Aug 21, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
There is a lot of redundancy in headers for Braid subscriptions:
Content-Type
andMerge-Type
Patch-Type
Example:
This has been cropping up in other issues, notably: #106
I currently see three ways to address this.
(1.) Updates could inherit the headers from previous updates by default, and then override any that should be different.
(2.) Patches could inherit the headers from the higher-level updates. Specifically, this is the proposal in #106, which would let you specify the
Patch-Type
as a top-level header in a subscription, that all patches would inherit from by default.(3.) Use HTTP/2 header compression (called HPACK) to reduce redundancy in general.
In summary, this boils down to:
(Note that I'm presuming we're moving from a 3-level hierarchy to a 2-level hierarchy, as articulated here.)
The text was updated successfully, but these errors were encountered: