Skip to content

Commit 6aa2a57

Browse files
committed
docs(implementations): audit report
1 parent 2bcb175 commit 6aa2a57

File tree

12 files changed

+26
-66
lines changed

12 files changed

+26
-66
lines changed

implementations/apollo-server/README.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@
1313
<ol>
1414
<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>
16-
<li><code>47DE</code> SHOULD accept */* and use application/json for the content-type</li>
17-
<li><code>80D8</code> SHOULD assume application/json content-type when accept is missing</li>
16+
<li><code>47DE</code> SHOULD accept */* and use application/graphql-response+json or application/json for the content-type</li>
17+
<li><code>80D8</code> SHOULD assume application/json or application/graphql-response+json content-type when accept is missing</li>
1818
<li><code>82A3</code> MUST use utf-8 encoding when responding</li>
1919
<li><code>BF61</code> MUST accept utf-8 encoded request</li>
2020
<li><code>78D5</code> MUST assume utf-8 in request if encoding is unspecified</li>

implementations/deno/README.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@
1313
<h2>Passing</h2>
1414
<ol>
1515
<li><code>4655</code> MUST accept application/json and match the content-type</li>
16-
<li><code>47DE</code> SHOULD accept */* and use application/json for the content-type</li>
17-
<li><code>80D8</code> SHOULD assume application/json content-type when accept is missing</li>
16+
<li><code>47DE</code> SHOULD accept */* and use application/graphql-response+json or application/json for the content-type</li>
17+
<li><code>80D8</code> SHOULD assume application/json or application/graphql-response+json content-type when accept is missing</li>
1818
<li><code>82A3</code> MUST use utf-8 encoding when responding</li>
1919
<li><code>BF61</code> MUST accept utf-8 encoded request</li>
2020
<li><code>78D5</code> MUST assume utf-8 in request if encoding is unspecified</li>

implementations/express-graphql/README.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@
1313
<h2>Passing</h2>
1414
<ol>
1515
<li><code>4655</code> MUST accept application/json and match the content-type</li>
16-
<li><code>47DE</code> SHOULD accept */* and use application/json for the content-type</li>
17-
<li><code>80D8</code> SHOULD assume application/json content-type when accept is missing</li>
16+
<li><code>47DE</code> SHOULD accept */* and use application/graphql-response+json or application/json for the content-type</li>
17+
<li><code>80D8</code> SHOULD assume application/json or application/graphql-response+json content-type when accept is missing</li>
1818
<li><code>82A3</code> MUST use utf-8 encoding when responding</li>
1919
<li><code>BF61</code> MUST accept utf-8 encoded request</li>
2020
<li><code>78D5</code> MUST assume utf-8 in request if encoding is unspecified</li>

implementations/graph-client/README.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,8 @@
1212
<ol>
1313
<li><code>22EB</code> MUST accept application/graphql-response+json and match the content-type</li>
1414
<li><code>4655</code> MUST accept application/json and match the content-type</li>
15-
<li><code>47DE</code> SHOULD accept */* and use application/json for the content-type</li>
16-
<li><code>80D8</code> SHOULD assume application/json content-type when accept is missing</li>
15+
<li><code>47DE</code> SHOULD accept */* and use application/graphql-response+json or application/json for the content-type</li>
16+
<li><code>80D8</code> SHOULD assume application/json or application/graphql-response+json content-type when accept is missing</li>
1717
<li><code>82A3</code> MUST use utf-8 encoding when responding</li>
1818
<li><code>BF61</code> MUST accept utf-8 encoded request</li>
1919
<li><code>78D5</code> MUST assume utf-8 in request if encoding is unspecified</li>

implementations/graphql-helix/README.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@
1313
<h2>Passing</h2>
1414
<ol>
1515
<li><code>4655</code> MUST accept application/json and match the content-type</li>
16-
<li><code>47DE</code> SHOULD accept */* and use application/json for the content-type</li>
17-
<li><code>80D8</code> SHOULD assume application/json content-type when accept is missing</li>
16+
<li><code>47DE</code> SHOULD accept */* and use application/graphql-response+json or application/json for the content-type</li>
17+
<li><code>80D8</code> SHOULD assume application/json or application/graphql-response+json content-type when accept is missing</li>
1818
<li><code>82A3</code> MUST use utf-8 encoding when responding</li>
1919
<li><code>BF61</code> MUST accept utf-8 encoded request</li>
2020
<li><code>78D5</code> MUST assume utf-8 in request if encoding is unspecified</li>

implementations/graphql-yoga/README.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,8 @@
1111
<ol>
1212
<li><code>22EB</code> MUST accept application/graphql-response+json and match the content-type</li>
1313
<li><code>4655</code> MUST accept application/json and match the content-type</li>
14-
<li><code>47DE</code> SHOULD accept */* and use application/json for the content-type</li>
15-
<li><code>80D8</code> SHOULD assume application/json content-type when accept is missing</li>
14+
<li><code>47DE</code> SHOULD accept */* and use application/graphql-response+json or application/json for the content-type</li>
15+
<li><code>80D8</code> SHOULD assume application/json or application/graphql-response+json content-type when accept is missing</li>
1616
<li><code>82A3</code> MUST use utf-8 encoding when responding</li>
1717
<li><code>BF61</code> MUST accept utf-8 encoded request</li>
1818
<li><code>78D5</code> MUST assume utf-8 in request if encoding is unspecified</li>

implementations/hotchocolate/README.md

+4-44
Original file line numberDiff line numberDiff line change
@@ -4,14 +4,16 @@
44

55
<ul>
66
<li><b>68</b> audits in total</li>
7-
<li><span style="font-family: monospace">✅</span> <b>62</b> pass</li>
8-
<li><span style="font-family: monospace">❗️</span> <b>6</b> warnings (optional)</li>
7+
<li><span style="font-family: monospace">✅</span> <b>64</b> pass</li>
8+
<li><span style="font-family: monospace">❗️</span> <b>4</b> warnings (optional)</li>
99
</ul>
1010

1111
<h2>Passing</h2>
1212
<ol>
1313
<li><code>22EB</code> MUST accept application/graphql-response+json and match the content-type</li>
1414
<li><code>4655</code> MUST accept application/json and match the content-type</li>
15+
<li><code>47DE</code> SHOULD accept */* and use application/graphql-response+json or application/json for the content-type</li>
16+
<li><code>80D8</code> SHOULD assume application/json or application/graphql-response+json content-type when accept is missing</li>
1517
<li><code>82A3</code> MUST use utf-8 encoding when responding</li>
1618
<li><code>BF61</code> MUST accept utf-8 encoded request</li>
1719
<li><code>78D5</code> MUST assume utf-8 in request if encoding is unspecified</li>
@@ -77,48 +79,6 @@
7779
<h2>Warnings</h2>
7880
The server <i>SHOULD</i> support these, but is not required.
7981
<ol>
80-
<li><code>47DE</code> SHOULD accept */* and use application/json for the content-type
81-
<details>
82-
<summary>Response header content-type does not contain application/json</summary>
83-
<pre><code class="lang-json">{
84-
"statusText": "OK",
85-
"status": 200,
86-
"headers": {
87-
"transfer-encoding": "chunked",
88-
"server": "Kestrel",
89-
"date": "<timestamp>",
90-
"content-type": "application/graphql-response+json; charset=utf-8"
91-
},
92-
"body": {
93-
"data": {
94-
"__typename": "Query"
95-
}
96-
}
97-
}
98-
</code></pre>
99-
</details>
100-
</li>
101-
<li><code>80D8</code> SHOULD assume application/json content-type when accept is missing
102-
<details>
103-
<summary>Response header content-type does not contain application/json</summary>
104-
<pre><code class="lang-json">{
105-
"statusText": "OK",
106-
"status": 200,
107-
"headers": {
108-
"transfer-encoding": "chunked",
109-
"server": "Kestrel",
110-
"date": "<timestamp>",
111-
"content-type": "application/graphql-response+json; charset=utf-8"
112-
},
113-
"body": {
114-
"data": {
115-
"__typename": "Query"
116-
}
117-
}
118-
}
119-
</code></pre>
120-
</details>
121-
</li>
12282
<li><code>58B0</code> SHOULD use 4xx or 5xx status codes on string {extensions} parameter when accepting application/json
12383
<details>
12484
<summary>Response status is not between 400 and 599</summary>
+2-2
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{
22
"total": 68,
3-
"ok": 62,
3+
"ok": 64,
44
"notice": 0,
5-
"warn": 6,
5+
"warn": 4,
66
"error": 0
77
}

implementations/lighthouse/README.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@
1313
<h2>Passing</h2>
1414
<ol>
1515
<li><code>4655</code> MUST accept application/json and match the content-type</li>
16-
<li><code>47DE</code> SHOULD accept */* and use application/json for the content-type</li>
17-
<li><code>80D8</code> SHOULD assume application/json content-type when accept is missing</li>
16+
<li><code>47DE</code> SHOULD accept */* and use application/graphql-response+json or application/json for the content-type</li>
17+
<li><code>80D8</code> SHOULD assume application/json or application/graphql-response+json content-type when accept is missing</li>
1818
<li><code>82A3</code> MUST use utf-8 encoding when responding</li>
1919
<li><code>BF61</code> MUST accept utf-8 encoded request</li>
2020
<li><code>78D5</code> MUST assume utf-8 in request if encoding is unspecified</li>

implementations/mercurius/README.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@
1313
<h2>Passing</h2>
1414
<ol>
1515
<li><code>4655</code> MUST accept application/json and match the content-type</li>
16-
<li><code>47DE</code> SHOULD accept */* and use application/json for the content-type</li>
17-
<li><code>80D8</code> SHOULD assume application/json content-type when accept is missing</li>
16+
<li><code>47DE</code> SHOULD accept */* and use application/graphql-response+json or application/json for the content-type</li>
17+
<li><code>80D8</code> SHOULD assume application/json or application/graphql-response+json content-type when accept is missing</li>
1818
<li><code>82A3</code> MUST use utf-8 encoding when responding</li>
1919
<li><code>BF61</code> MUST accept utf-8 encoded request</li>
2020
<li><code>78D5</code> MUST assume utf-8 in request if encoding is unspecified</li>

implementations/postgraphile/README.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@
1313
<h2>Passing</h2>
1414
<ol>
1515
<li><code>4655</code> MUST accept application/json and match the content-type</li>
16-
<li><code>47DE</code> SHOULD accept */* and use application/json for the content-type</li>
17-
<li><code>80D8</code> SHOULD assume application/json content-type when accept is missing</li>
16+
<li><code>47DE</code> SHOULD accept */* and use application/graphql-response+json or application/json for the content-type</li>
17+
<li><code>80D8</code> SHOULD assume application/json or application/graphql-response+json content-type when accept is missing</li>
1818
<li><code>82A3</code> MUST use utf-8 encoding when responding</li>
1919
<li><code>BF61</code> MUST accept utf-8 encoded request</li>
2020
<li><code>78D5</code> MUST assume utf-8 in request if encoding is unspecified</li>

implementations/thegraph/README.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@
1313
<h2>Passing</h2>
1414
<ol>
1515
<li><code>4655</code> MUST accept application/json and match the content-type</li>
16-
<li><code>47DE</code> SHOULD accept */* and use application/json for the content-type</li>
17-
<li><code>80D8</code> SHOULD assume application/json content-type when accept is missing</li>
16+
<li><code>47DE</code> SHOULD accept */* and use application/graphql-response+json or application/json for the content-type</li>
17+
<li><code>80D8</code> SHOULD assume application/json or application/graphql-response+json content-type when accept is missing</li>
1818
<li><code>82A3</code> MUST use utf-8 encoding when responding</li>
1919
<li><code>BF61</code> MUST accept utf-8 encoded request</li>
2020
<li><code>78D5</code> MUST assume utf-8 in request if encoding is unspecified</li>

0 commit comments

Comments
 (0)