Skip to content

Commit 78c9bdb

Browse files
committed
Merge branch '2.3'
2 parents 0e76323 + f12aa2a commit 78c9bdb

File tree

3 files changed

+7
-2
lines changed

3 files changed

+7
-2
lines changed

CHANGELOG.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@
3434

3535
## 2.3.6
3636

37+
* /!\ Security: a vulnerability impacting the GraphQL subsystem was allowing users authorized to run mutations for a specific resource type, to execute it on any resource, of any type
3738
* Fix normalization of raw collections (not API resources)
3839
* Fix content negotiation format matching
3940

@@ -116,6 +117,10 @@
116117
* GraphQL: Add a `totalCount` field in GraphQL paginated collections
117118
* JSONAPI: Allow inclusion of related resources
118119

120+
## 2.2.10
121+
122+
* /!\ Security: a vulnerability impacting the GraphQL subsystem was allowing users authorized to run mutations for a specific resource type, to execute it on any resource, of any type
123+
119124
## 2.2.9
120125

121126
* Fix `ExistsFilter` for inverse side of OneToOne association

features/graphql/mutation.feature

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ Feature: GraphQL mutation support
135135
Then the response status code should be 200
136136
And the response should be in JSON
137137
And the header "Content-Type" should be equal to "application/json"
138-
And the JSON node "errors[0].message" should be equal to 'Item "/dummies/1" did not match expected type "ApiPlatform\Core\Tests\Fixtures\TestBundle\Entity\Foo".'
138+
And the JSON node "errors[0].message" should be equal to 'Item "/dummies/1" did not match expected type "Foo".'
139139

140140
Scenario: Delete an item with composite identifiers through a mutation
141141
Given there are Composite identifier objects

src/GraphQl/Resolver/Factory/ItemMutationResolverFactory.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ public function __invoke(string $resourceClass = null, string $rootClass = null,
8787
}
8888

8989
if ($resourceClass !== $this->getObjectClass($item)) {
90-
throw Error::createLocatedError(sprintf('Item "%s" did not match expected type "%s".', $args['input']['id'], $resourceClass), $info->fieldNodes, $info->path);
90+
throw Error::createLocatedError(sprintf('Item "%s" did not match expected type "%s".', $args['input']['id'], $resourceMetadata->getShortName()), $info->fieldNodes, $info->path);
9191
}
9292
}
9393

0 commit comments

Comments
 (0)