Reject invalid directives on scalars in 15.0.0
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 85/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- graphql, kotlin
- Lĩnh vực
- backend-api-design
Hướng nghiên cứu
Bắt đầu tại SchemaParser#createScalar và kiểm tra cách các chỉ thị scalar không hợp lệ hiện được ghi lại và bỏ qua. Cập nhật DirectiveTest.should warn about undeclared directives on scalars and build them without the schema's directives để xác nhận một SchemaError, sau đó thêm thay đổi này vào ghi chú phát hành 15.0.0. Hoàn tất khi bài kiểm tra lỗi scalar và ghi chú phát hành phản ánh hành vi mới.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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
- Ngôn ngữ chính
- Kotlin
- Star
- 824
- Fork
- 175
- Merge trung bình
- 2 ngày 5 giờ
- Pull request đã merge (30 ngày)
- 52
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của graphql-java-kickstart/graphql-java-tools
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 72/100
graphql-java-kickstart/graphql-java-tools#889 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
graphql-java-kickstart/graphql-java-tools#886 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Reject directives applied at the wrong location, without a required argument or twice in 15.0.0Đang mởenhancement
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 72/100
graphql-java-kickstart/graphql-java-tools#884 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
graphql-java-kickstart/graphql-java-tools#881 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Directive wirings see a type reference on recursive input fieldsCó thể đã có người làm @oryan-block đã nhận hôm nay. Đang mởbug
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
graphql-java-kickstart/graphql-java-tools#875 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của graphql-java-kickstart/graphql-java-tools
Issue tương tự
-
area/kotlin
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 1 ngày
-
good first issue help wanted
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100
dafonton77-tech/Cadreenf#11 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 80/100
Hy4ri/hermes-mobile#1647 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100
rawnaldclark/Stash#649 ·
Maintainer thường phản hồi trong vòng 1 ngày