GHSA-733v-p3h5-qpq7 json
GraphQL Armor Cost-Limit Plugin Bypass via Introspection Query Obfuscation
published last yr. modified last yr.
- cvss
- 5.3
- version 3.0, GitHub
- cisa kev
- Not listed
- not in CISA's catalog
- epss
- —
- no score, EPSS covers CVEs only
- public exploits
- 0
- none known
Summary
### Summary A query cost restriction using the `cost-limit` can be bypassed if `ignoreIntrospection` is enabled (which is the default configuration) by naming your query/fragment `__schema`.
### Details At the start of the `computeComplexity` function, we have the following check for `ignoreIntrospection` option:
```ts if (this.config.ignoreIntrospection && 'name' in node && node.name?.value === '__schema') { return 0; } ```
However, the `node` can be `FieldNode | FragmentDefinitionNode | InlineFragmentNode | OperationDefinitionNode | FragmentSpreadNode`
So, for example, sending the following query
```gql query hello { books { title } } ```
would create an `OperationDefinitionNode` with `node.name.value == 'hello'`
The proper way to handle this would be to check for the `__schema` field, which would create a `FieldNode`.
The fix is
```ts if ( this.config.ignoreIntrospection && 'name' in node && node.name?.value === '__schema' && node.kind === Kind.FIELD ) { return 0; } ```
to assert that the node must be a `FieldNode`
### PoC ```gql query { ...__schema }
fragment __schema on Query { books { title author } } ```
```gql query __schema { books { title author } } ```
### Impact Applications using GraphQL Armor Cost Limit plugin with `ignoreIntrospection` enabled.
### Fix: Fixed on [772](https://github.com/Escape-Technologies/graphql-armor/pull/772). A quick patch would be to set `ignoreIntrospection` to false.
Patch / fix
- npm @escape.tech/graphql-armor-cost-limit 2.4.2
Is GHSA-733v-p3h5-qpq7 in your dependencies? Fencer shows which of your repositories are affected as soon as an advisory lands, and opens the fix as a pull request.
Check with FencerCVSS scores
| assigned by | version | score | vector |
|---|---|---|---|
| GitHub | 3.0 | 5.3 | CVSS:3.0/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:N/A:L |
Affected packages
| ecosystem | package | affected | fixed in |
|---|---|---|---|
| npm | @escape.tech/graphql-armor-cost-limit | <2.4.2 |
2.4.2 |
Classification
References
-
github.com
/Escape-Technologies/graphql-armor/pull/772
web
-
github.com
/Escape-Technologies/graphql-armor/commit/5a329541cf32a359ee1f69748738f91231b26eba
web
-
github.com
/Escape-Technologies/graphql-armor/security/advisories/GHSA-733v-p3h5-qpq7
web
-
github.com
/Escape-Technologies/graphql-armor
package
Sources
- GitHub Advisory GHSA-733v-p3h5-qpq7
Fencer Intel keeps one entry per vulnerability: the CVE id wins, then GHSA, and records that alias each other are merged, except that two CVEs always keep separate entries. The same data is available as JSON.