Commit 6d97f25
committed
feat: pass abortSignal to resolvers via GraphQLResolveInfo
In graphql#4261 (not yet released in v17) we made abortSignal available to
resolvers via a fifth argument to the field resolver. Among other
things, this means that any code that processes schemas to wrap
resolvers in other functions would have to be aware of this one new
feature and specially thread through the new behavior. It also changed
the TypeScript signature of GraphQLFieldResolver to *require* passing
the fifth argument (even if undefined).
But the field resolver interface already has a place for GraphQL-JS to
put a grab-bag of helpful named objects for use by resolvers:
`GraphQLResolveInfo`.
This PR (which is not backwards compatible with v17.0.0-alpha.8, but is
backwards-compatible with v16) moves the abortSignal into
`GraphQLResolveInfo`.1 parent f39fefb commit 6d97f25
File tree
5 files changed
+13
-8
lines changed- src
- execution
- __tests__
- type
- website/pages/upgrade-guides
5 files changed
+13
-8
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
151 | 151 | | |
152 | 152 | | |
153 | 153 | | |
154 | | - | |
155 | | - | |
| 154 | + | |
| 155 | + | |
156 | 156 | | |
157 | 157 | | |
158 | 158 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
213 | 213 | | |
214 | 214 | | |
215 | 215 | | |
| 216 | + | |
216 | 217 | | |
217 | 218 | | |
218 | 219 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
875 | 875 | | |
876 | 876 | | |
877 | 877 | | |
| 878 | + | |
878 | 879 | | |
879 | 880 | | |
880 | 881 | | |
| |||
893 | 894 | | |
894 | 895 | | |
895 | 896 | | |
896 | | - | |
| 897 | + | |
897 | 898 | | |
898 | 899 | | |
899 | 900 | | |
| |||
960 | 961 | | |
961 | 962 | | |
962 | 963 | | |
| 964 | + | |
963 | 965 | | |
964 | 966 | | |
965 | 967 | | |
| |||
976 | 978 | | |
977 | 979 | | |
978 | 980 | | |
| 981 | + | |
979 | 982 | | |
980 | 983 | | |
981 | 984 | | |
| |||
2079 | 2082 | | |
2080 | 2083 | | |
2081 | 2084 | | |
2082 | | - | |
| 2085 | + | |
2083 | 2086 | | |
2084 | 2087 | | |
2085 | 2088 | | |
2086 | 2089 | | |
2087 | | - | |
| 2090 | + | |
2088 | 2091 | | |
2089 | 2092 | | |
2090 | 2093 | | |
| |||
2293 | 2296 | | |
2294 | 2297 | | |
2295 | 2298 | | |
| 2299 | + | |
2296 | 2300 | | |
2297 | 2301 | | |
2298 | 2302 | | |
| |||
2317 | 2321 | | |
2318 | 2322 | | |
2319 | 2323 | | |
2320 | | - | |
| 2324 | + | |
2321 | 2325 | | |
2322 | 2326 | | |
2323 | 2327 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
997 | 997 | | |
998 | 998 | | |
999 | 999 | | |
1000 | | - | |
1001 | 1000 | | |
1002 | 1001 | | |
1003 | 1002 | | |
| |||
1011 | 1010 | | |
1012 | 1011 | | |
1013 | 1012 | | |
| 1013 | + | |
1014 | 1014 | | |
1015 | 1015 | | |
1016 | 1016 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
178 | 178 | | |
179 | 179 | | |
180 | 180 | | |
181 | | - | |
| 181 | + | |
182 | 182 | | |
183 | 183 | | |
184 | 184 | | |
| |||
0 commit comments