Skip to content

Remove separation between HTTP GET response headers and first version's headers#84

Closed
canadaduane wants to merge 1 commit intobraid-org:masterfrom
canadaduane:get-first-version
Closed

Remove separation between HTTP GET response headers and first version's headers#84
canadaduane wants to merge 1 commit intobraid-org:masterfrom
canadaduane:get-first-version

Conversation

@canadaduane
Copy link
Member

@canadaduane canadaduane commented Feb 9, 2021

Status: This is a proposal meant for discussion, not a request to merge.

@josephg
Copy link
Contributor

josephg commented Mar 25, 2021

This issue was raised during meeting 6. This change will cause errors if we send a top level Content-Length header in a response, because the client will assume that will frame the entire response.

We might still be able to do this, but we should discuss it after #97 has been resolved - which is out of scope for milestone 03.

@josephg josephg removed this from the Braid-HTTP 03 milestone Mar 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants