Skip to content

Commit 9692169

Browse files
authored
IncrementalDelivery: clarify exactly where CRLF is (#219)
1 parent b684991 commit 9692169

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

rfcs/IncrementalDelivery.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ The HTTP response for an incrementally delivered response should conform to the
1717
An example response body will look like:
1818

1919
```
20+
2021
---
2122
Content-Type: application/json; charset=utf-8
2223
@@ -28,11 +29,10 @@ Content-Type: application/json; charset=utf-8
2829
-----
2930
```
3031
* The boundary used is `-` and is passed to the client in the http response's `Content-Type` header. Note that headers can appear in both the HTTP response itself and as part of the response body. The `Content-Type` header must be sent in the HTTP response.
31-
* An initial boundary is sent marking the end of the preamble area.
32+
* Before each part of the multi-part response, a boundary (`CRLF`, `---`, `CRLF`) is sent.
3233
* Each part of the multipart response must contain a `Content-Type` header. Similar to the GraphQL specification this specification does not require a specific serialization format. For consistency and ease of notation, examples of the response are given in JSON throughout the spec.
33-
* After all headers, an additional `CRLF` is sent.
34-
* The payload is sent, followed by a `CRLF`.
35-
* After each payload, a boundary is sent. For the final payload, the terminating boundary of `-----` followed by a `CRLF` is sent. For all other payloads a boundary of `---` followed by a `CRFL` is sent.
34+
* After all headers for each part, an additional `CRLF` is sent, followed by the payload for the part.
35+
* After the final payload, the terminating boundary of `CRLF` followed by `-----` followed by `CRLF` is sent.
3636

3737
## Server Implementations
3838
* `express-graphql`: [pull request](https://github.com/graphql/express-graphql/pull/583)

0 commit comments

Comments
 (0)