Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat(graphql): allow to disable the introspection query #5711

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
alanpoulain merged 13 commits into from
Aug 8, 2023
Merged

feat(graphql): allow to disable the introspection query #5711

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
3e5f8a5
feat(graphql): allow to enable/disable the introspection query
epourail Aug 2, 2023
be8bc99
feat(graphql): allow to enable/disable the introspection query
epourail Aug 2, 2023
ad911b7
feat(graphql): allow to enable/disable the introspection query
epourail Aug 3, 2023
0736529
fix tests
epourail Aug 4, 2023
adc879f
add unitary tests
epourail Aug 4, 2023
bfff3f3
fix php-cs-fixer
epourail Aug 4, 2023
d161ef8
fix
epourail Aug 4, 2023
36b8672
fix
epourail Aug 4, 2023
b553aae
cleanup
epourail Aug 4, 2023
541d0a9
fix unit tests
epourail Aug 8, 2023
0cdc49f
fix php-cs-fixer
epourail Aug 8, 2023
a325b6c
remove unuseful tests
epourail Aug 8, 2023
9880458
tests
epourail Aug 8, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
feat(graphql): allow to enable/disable the introspection query
  • Loading branch information
@epourail
epourail committed Aug 8, 2023
commit ad911b7dcda50491ec52cdcd0d1181dc8ed3096f
12 changes: 9 additions & 3 deletions src/GraphQl/Executor.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
use GraphQL\Executor\ExecutionResult;
use GraphQL\GraphQL;
use GraphQL\Type\Schema;
use GraphQL\Validator\DocumentValidator;
use GraphQL\Validator\Rules\DisableIntrospection;

/**
Expand All @@ -27,16 +28,21 @@ final class Executor implements ExecutorInterface
{
public function __construct(private readonly bool $graphQlIntrospectionEnabled = true)
{
DocumentValidator::addRule(
new DisableIntrospection(
$this->graphQlIntrospectionEnabled ? DisableIntrospection::DISABLED : DisableIntrospection::ENABLED
)
);
}

/**
* {@inheritdoc}
*/
public function executeQuery(Schema $schema, $source, mixed $rootValue = null, mixed $context = null, array $variableValues = null, string $operationName = null, callable $fieldResolver = null, array $validationRules = null): ExecutionResult
{
$validationRules[] = new DisableIntrospection(
$this->graphQlIntrospectionEnabled ? DisableIntrospection::DISABLED : DisableIntrospection::ENABLED
);
// $validationRules[] = new DisableIntrospection(
// $this->graphQlIntrospectionEnabled ? DisableIntrospection::DISABLED : DisableIntrospection::ENABLED
// );

return GraphQL::executeQuery($schema, $source, $rootValue, $context, $variableValues, $operationName, $fieldResolver, $validationRules);
}
Expand Down