|
| 1 | +# GraphQL-over-HTTP WG - 26th October 2023 |
| 2 | + |
| 3 | +**Watch the replay**: |
| 4 | +https://www.youtube.com/watch?v=nHSixplvCc0&list=PLP1igyLx8foEz9127xc0SsabIrbTMt9g5 |
| 5 | + |
| 6 | +## Agenda |
| 7 | + |
| 8 | +- Accept header with unsupported formats |
| 9 | +- Persisted Operations |
| 10 | +- Advancing GraphQL over HTTP to stage 2 |
| 11 | + |
| 12 | +## Accept header PR |
| 13 | + |
| 14 | +Linking back to the HTTP specification, either respond with non-acceptable when |
| 15 | +we can’t serve the format in the Accept header (recommended), not recommended |
| 16 | +would be to answer in an unaccepted format. |
| 17 | + |
| 18 | +- PR from Benjie |
| 19 | + [https://github.com/graphql/graphql-over-http/pull/227](https://github.com/graphql/graphql-over-http/pull/227) |
| 20 | + |
| 21 | +General approval, no reservations - will merge. |
| 22 | + |
| 23 | +## Persisted Operations RFC |
| 24 | + |
| 25 | +Common pattern in Facebook since before GraphQL was open-sourced, this was used |
| 26 | +to enhance security so only recognised operations can be executed. Worth calling |
| 27 | +out that currently there is no specification for persisted operations, hash it |
| 28 | +and send it to the server is the current state. |
| 29 | + |
| 30 | +- RFC from Jovi |
| 31 | + [https://github.com/graphql/graphql-over-http/blob/main/rfcs/PersistedOperations.md](https://github.com/graphql/graphql-over-http/blob/main/rfcs/PersistedOperations.md) |
| 32 | +- RFC w/ Appendix from Benjie |
| 33 | + [https://github.com/graphql/graphql-over-http/pull/264](https://github.com/graphql/graphql-over-http/pull/264) |
| 34 | + |
| 35 | +APQ can be extended on top of the RFC’s but there’s some missing pieces as we |
| 36 | +wouldn’t want to encourage using the extensions object for that as these are |
| 37 | +free from the spec. |
| 38 | + |
| 39 | +Relay would be compatible as it has its own network layer, chosen not to advance |
| 40 | +this RFC this close to advancing the HTTP spec. |
| 41 | + |
| 42 | +Coupling back to the current spec, `query` is mandatory, so from a compatibility |
| 43 | +point persisted documents wouldn’t be compliant? It’s modeled differently, |
| 44 | +optional to implement and it’s a translation to a query. |
| 45 | + |
| 46 | +## Advancing the spec |
| 47 | + |
| 48 | +Denis: been testing a lot of implementations, some are behind but shouldn’t be |
| 49 | +too much effort to get them in line. |
| 50 | + |
| 51 | +→ advancing the GraphQL Over HTTP spec to stage 2 |
| 52 | + |
| 53 | +Everyone is already treating this as a stage 2 spec, actually going to stage 3 |
| 54 | +will involve some legal work, getting signatures from contributors, … Help from |
| 55 | +the Linux Foundation would happen here. TSC will vote before releasing the 1.0 |
| 56 | +version. |
| 57 | + |
| 58 | +**Action item**: Benjie will create a pull request and leave it open for a few |
| 59 | +weeks so folks can express any concerns. |
| 60 | + |
| 61 | +**Action item**: Benjie to add a 3 minute agenda item to the core WG meeting |
0 commit comments