Описание
GraphQL grant on a property might be cached with different objects
Original message:
I found an issue with security grants on on properties in the GraphQL ItemNormalizer:
If you use something like #[ApiProperty(security: 'is_granted("PROPERTY_READ", [object, property])')] on a member of an entity, the grant gets cached and is only evaluated once, even if the object in question is a different one.
There is the ApiPlatform\GraphQl\Serializer\ItemNormalizer::isCacheKeySafe() method that seems to be intended to prevent this: https://github.com/api-platform/core/blob/88f5ac50d20d6510686a7552310cc567fcca45bf/src/GraphQl/Serializer/ItemNormalizer.php#L160-L164
and in its usage on line 90 it does indeed not create a cache key, but the parent::normalize() that is called afterwards still creates the cache key and causes the issue.
Impact
It grants access to properties that it should not.
Workarounds
Override the ItemNormalizer.
Patched at: https://github.com/api-platform/core/commit/7af65aad13037d7649348ee3dcd88e084ef771f8
Ссылки
- https://github.com/api-platform/core/security/advisories/GHSA-428q-q3vv-3fq3
- https://nvd.nist.gov/vuln/detail/CVE-2025-31485
- https://github.com/api-platform/core/commit/7af65aad13037d7649348ee3dcd88e084ef771f8
- https://github.com/api-platform/core/commit/cba3acfbd517763cf320167250c5bed6d569696a
- https://github.com/FriendsOfPHP/security-advisories/blob/master/api-platform/core/CVE-2025-31485.yaml
- https://github.com/FriendsOfPHP/security-advisories/blob/master/api-platform/graphql/CVE-2025-31485.yaml
- https://github.com/api-platform/core/releases/tag/v3.4.17
Пакеты
api-platform/graphql
>= 4.0.0-alpha.1, < 4.0.22
4.0.22
api-platform/core
>= 4.0.0-alpha.1, < 4.0.22
4.0.22
api-platform/graphql
< 3.4.17
3.4.17
api-platform/core
< 3.4.17
3.4.17
api-platform/core
>= 4.1.0-alpha.1, < 4.1.5
4.1.5
api-platform/graphql
>= 4.1.0-alpha.1, < 4.1.5
4.1.5
Связанные уязвимости
API Platform Core is a system to create hypermedia-driven REST and GraphQL APIs. Prior to 4.0.22 and 3.4.17, a GraphQL grant on a property might be cached with different objects. The ApiPlatform\GraphQl\Serializer\ItemNormalizer::isCacheKeySafe() method is meant to prevent the caching but the parent::normalize method that is called afterwards still creates the cache key and causes the issue. This vulnerability is fixed in 4.0.22 and 3.4.17.