Reject invalid directives on scalars in 15.0.0
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 85/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- graphql, kotlin
- Bereich
- backend-api-design
Rechercherichtung
Beginne in SchemaParser#createScalar und untersuche, wie ungültige Scalar-Direktiven derzeit protokolliert und ausgelassen werden. Aktualisiere DirectiveTest.should warn about undeclared directives on scalars and build them without the schema's directives, sodass es einen SchemaError erwartet, und nimm die Änderung anschließend in die Release Notes für 15.0.0 auf. Fertig ist es, wenn der Scalar-Fehlertest und die Release Notes das neue Verhalten widerspiegeln.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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
- Vorherrschende Sprache
- Kotlin
- Sterne
- 824
- Forks
- 175
- Ø Merge
- 2 T. 5 Std.
- Gemergte PRs (30 T.)
- 52
Entwicklungsumgebung
Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus graphql-java-kickstart/graphql-java-tools
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 72/100
graphql-java-kickstart/graphql-java-tools#889 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 55/100
graphql-java-kickstart/graphql-java-tools#886 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Reject directives applied at the wrong location, without a required argument or twice in 15.0.0Offenenhancement
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 72/100
graphql-java-kickstart/graphql-java-tools#884 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
graphql-java-kickstart/graphql-java-tools#881 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Directive wirings see a type reference on recursive input fieldsEvtl. vergeben @oryan-block hat das heute übernommen. Offenbug
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
graphql-java-kickstart/graphql-java-tools#875 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in graphql-java-kickstart/graphql-java-tools
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Maintainer antworten meist innerhalb von 1 Tag
-
good first issue help wanted
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 66/100
dafonton77-tech/Cadreenf#11 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 80/100
Hy4ri/hermes-mobile#1647 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 67/100
rawnaldclark/Stash#649 ·
Maintainer antworten meist innerhalb von 1 Tag
-
documentation
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 79/100
Maintainer antworten meist innerhalb von 1 Tag