Skip to content

Commit e403aae

Browse files
committed
docs(implementations): audit report
1 parent 5f5532b commit e403aae

File tree

19 files changed

+772
-751
lines changed

19 files changed

+772
-751
lines changed

README.md

-4
Original file line numberDiff line numberDiff line change
@@ -890,14 +890,10 @@ Their compliance with the [GraphQL over HTTP spec](https://graphql.github.io/gra
890890
| Name | Audit |
891891
|------|-------|
892892
| [apollo-server](https://www.apollographql.com/docs/apollo-server) | [✅ Compliant](/implementations/apollo-server/README.md) |
893-
| [deno](https://deno.com/blog/build-a-graphql-server-with-deno) | [✅ Compliant](/implementations/deno/README.md) |
894893
| [graph-client](https://github.com/graphprotocol/graph-client) | [✅ Compliant](/implementations/graph-client/README.md) |
895-
| [graphql-helix](https://www.graphql-helix.com) | [✅ Compliant](/implementations/graphql-helix/README.md) |
896894
| [graphql-yoga](https://www.the-guild.dev/graphql/yoga-server) | [✅ Compliant](/implementations/graphql-yoga/README.md) |
897895
| [hotchocolate](https://chillicream.com/docs/hotchocolate) | [✅ Compliant](/implementations/hotchocolate/README.md) |
898-
| [lighthouse](https://lighthouse-php.com) | [✅ Compliant](/implementations/lighthouse/README.md) |
899896
| [pioneer](https://pioneer.dexclaimation.com) | [✅ Compliant](/implementations/pioneer/README.md) |
900-
| [postgraphile](https://www.graphile.org/postgraphile) | [✅ Compliant](/implementations/postgraphile/README.md) |
901897
<!-- prettier-ignore-end -->
902898

903899
<!-- </ServersTable> -->

implementations/apollo-server/README.md

+14-14
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111

1212
<h2>Passing</h2>
1313
<ol>
14-
<li><code>22EB</code> SHOULD accept application/graphql-response+json and match the content-type</li>
14+
<li><code>22EB</code> MUST accept application/graphql-response+json and match the content-type</li>
1515
<li><code>4655</code> MUST accept application/json and match the content-type</li>
1616
<li><code>47DE</code> SHOULD accept */* and use application/json for the content-type</li>
1717
<li><code>80D8</code> SHOULD assume application/json content-type when accept is missing</li>
@@ -28,40 +28,40 @@
2828
<li><code>LKJ1</code> MAY use 400 status code on number {query} parameter</li>
2929
<li><code>LKJ2</code> MAY use 400 status code on boolean {query} parameter</li>
3030
<li><code>LKJ3</code> MAY use 400 status code on array {query} parameter</li>
31-
<li><code>34A2</code> SHOULD allow string {query} parameter when accepting application/graphql-response+json</li>
31+
<li><code>34A2</code> MUST allow string {query} parameter when accepting application/graphql-response+json</li>
3232
<li><code>13EE</code> MUST allow string {query} parameter when accepting application/json</li>
3333
<li><code>6C00</code> MAY use 400 status code on object {operationName} parameter</li>
3434
<li><code>6C01</code> MAY use 400 status code on number {operationName} parameter</li>
3535
<li><code>6C02</code> MAY use 400 status code on boolean {operationName} parameter</li>
3636
<li><code>6C03</code> MAY use 400 status code on array {operationName} parameter</li>
37-
<li><code>8161</code> SHOULD allow string {operationName} parameter when accepting application/graphql-response+json</li>
37+
<li><code>8161</code> MUST allow string {operationName} parameter when accepting application/graphql-response+json</li>
3838
<li><code>B8B3</code> MUST allow string {operationName} parameter when accepting application/json</li>
39-
<li><code>94B0</code> SHOULD allow null {variables} parameter when accepting application/graphql-response+json</li>
39+
<li><code>94B0</code> MUST allow null {variables} parameter when accepting application/graphql-response+json</li>
4040
<li><code>0220</code> MUST allow null {variables} parameter when accepting application/json</li>
41-
<li><code>94B1</code> SHOULD allow null {operationName} parameter when accepting application/graphql-response+json</li>
41+
<li><code>94B1</code> MUST allow null {operationName} parameter when accepting application/graphql-response+json</li>
4242
<li><code>0221</code> MUST allow null {operationName} parameter when accepting application/json</li>
43-
<li><code>94B2</code> SHOULD allow null {extensions} parameter when accepting application/graphql-response+json</li>
43+
<li><code>94B2</code> MUST allow null {extensions} parameter when accepting application/graphql-response+json</li>
4444
<li><code>0222</code> MUST allow null {extensions} parameter when accepting application/json</li>
4545
<li><code>4760</code> MAY use 400 status code on string {variables} parameter</li>
4646
<li><code>4761</code> MAY use 400 status code on number {variables} parameter</li>
4747
<li><code>4762</code> MAY use 400 status code on boolean {variables} parameter</li>
4848
<li><code>4763</code> MAY use 400 status code on array {variables} parameter</li>
49-
<li><code>2EA1</code> SHOULD allow map {variables} parameter when accepting application/graphql-response+json</li>
49+
<li><code>2EA1</code> MUST allow map {variables} parameter when accepting application/graphql-response+json</li>
5050
<li><code>28B9</code> MUST allow map {variables} parameter when accepting application/json</li>
51-
<li><code>58B0</code> MAY use 400 status code on string {extensions} parameter</li>
52-
<li><code>58B1</code> MAY use 400 status code on number {extensions} parameter</li>
53-
<li><code>58B2</code> MAY use 400 status code on boolean {extensions} parameter</li>
54-
<li><code>58B3</code> MAY use 400 status code on array {extensions} parameter</li>
55-
<li><code>428F</code> SHOULD allow map {extensions} parameter when accepting application/graphql-response+json</li>
51+
<li><code>58B0</code> MUST use 400 status code on string {extensions} parameter</li>
52+
<li><code>58B1</code> MUST use 400 status code on number {extensions} parameter</li>
53+
<li><code>58B2</code> MUST use 400 status code on boolean {extensions} parameter</li>
54+
<li><code>58B3</code> MUST use 400 status code on array {extensions} parameter</li>
55+
<li><code>428F</code> MUST allow map {extensions} parameter when accepting application/graphql-response+json</li>
5656
<li><code>1B7A</code> MUST allow map {extensions} parameter when accepting application/json</li>
5757
<li><code>B6DC</code> MAY use 4xx or 5xx status codes on JSON parsing failure</li>
5858
<li><code>BCF8</code> MAY use 400 status code on JSON parsing failure</li>
5959
<li><code>8764</code> MAY use 4xx or 5xx status codes if parameters are invalid</li>
6060
<li><code>3E3A</code> MAY use 400 status code if parameters are invalid</li>
61-
<li><code>865D</code> SHOULD use 4xx or 5xx status codes on document parsing failure when accepting application/graphql-response+json</li>
61+
<li><code>865D</code> MUST use 4xx or 5xx status codes on document parsing failure when accepting application/graphql-response+json</li>
6262
<li><code>556A</code> SHOULD use 400 status code on document parsing failure when accepting application/graphql-response+json</li>
6363
<li><code>D586</code> SHOULD not contain the data entry on document parsing failure when accepting application/graphql-response+json</li>
64-
<li><code>51FE</code> SHOULD use 4xx or 5xx status codes on document validation failure when accepting application/graphql-response+json</li>
64+
<li><code>51FE</code> MUST use 4xx or 5xx status codes on document validation failure when accepting application/graphql-response+json</li>
6565
<li><code>74FF</code> SHOULD use 400 status code on document validation failure when accepting application/graphql-response+json</li>
6666
<li><code>5E5B</code> SHOULD not contain the data entry on document validation failure when accepting application/graphql-response+json</li>
6767
<li><code>86EE</code> SHOULD use a status code of 400 on variable coercion failure when accepting application/graphql-response+json</li>

0 commit comments

Comments
 (0)