Reject invalid directives on scalars in 15.0.0
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 85/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- graphql, kotlin
- Domaine
- backend-api-design
Piste de recherche
Commencez dans SchemaParser#createScalar et examinez comment les directives scalaires non valides sont actuellement consignées et omises. Modifiez DirectiveTest.should warn about undeclared directives on scalars and build them without the schema's directives pour vérifier qu'une SchemaError est levée, puis ajoutez la modification aux notes de version 15.0.0. Le travail est terminé lorsque le test d'erreur scalaire et les notes de version reflètent le nouveau comportement.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Since #864, scalars get the directives from the schema. When one of them is undeclared or has an argument its definition doesn't have, tools logs Scalar 'Date' is built without its directives from the schema: … and builds the scalar without them, because those scalars always built before. Every other type throws a SchemaError for the same mistake.
What to change
- throw the
SchemaErrorinSchemaParser#createScalarinstead of logging it - turn
DirectiveTest.should warn about undeclared directives on scalars and build them without the schema's directivesinto a test for the error - mention it in the 15.0.0 release notes
- Langage dominant
- Kotlin
- Étoiles
- 824
- Forks
- 175
- Merge moyen
- 2 j 5 h
- PR mergées (30 j)
- 52
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de graphql-java-kickstart/graphql-java-tools
-
Difficulté 3/5 1-2 jours Accessibilité débutants 72/100
graphql-java-kickstart/graphql-java-tools#889 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 3/5 1-2 jours Accessibilité débutants 55/100
graphql-java-kickstart/graphql-java-tools#886 ·
Les mainteneurs répondent en général sous 1 jour
-
Reject directives applied at the wrong location, without a required argument or twice in 15.0.0Ouverteenhancement
Difficulté 3/5 1-2 jours Accessibilité débutants 72/100
graphql-java-kickstart/graphql-java-tools#884 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
graphql-java-kickstart/graphql-java-tools#881 ·
Les mainteneurs répondent en général sous 1 jour
-
Directive wirings see a type reference on recursive input fieldsPeut-être pris @oryan-block l’a pris aujourd’hui. Ouvertebug
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
graphql-java-kickstart/graphql-java-tools#875 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de graphql-java-kickstart/graphql-java-tools
Issues similaires
-
area/kotlin
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour
-
good first issue help wanted
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
dafonton77-tech/Cadreenf#11 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 80/100
Hy4ri/hermes-mobile#1647 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 67/100
rawnaldclark/Stash#649 ·
Les mainteneurs répondent en général sous 1 jour