{#if spec.commonExpositions.type == "INGRESS"} never matches, so the gRPC TLS properties are never rendered
Đá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
- 58/100
Hướng nghiên cứu
Read MicrocksConfigMapDependentResource/application.properties around line 135 and the related conditions fixed in issue #306. Then inspect MicrocksConfigMapDependentResourceTest and add positive and negative cases for the default INGRESS type; done means both gRPC TLS properties appear for INGRESS and are absent for another type. The issue is unclaimed and has no linked pull request, but it follows work on #306.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Describe the bug
In MicrocksConfigMapDependentResource/application.properties L135, the gRPC TLS properties are guarded by {#if spec.commonExpositions.type == "INGRESS"}. type is the ExpositionType enum, not a String, and in Qute an enum compared with a String literal is never equal. This is the same issue fixed for the Kafka authentication conditions in #306 (.type → .type.name). Since INGRESS is the default exposition type, this affects default installs.
Expected behavior
With commonExpositions.type: INGRESS, the generated application.properties contains grpc.server.certChainFilePath and grpc.server.privateKeyFilePath.
Actual behavior
The two lines are never rendered, whatever the exposition type.
How to Reproduce?
Render MicrocksConfigMapDependentResource.desired() with the default spec (commonExpositions.type: INGRESS) in a @QuarkusTest: neither line is present. With L135 changed to spec.commonExpositions.type.name == "INGRESS", both are rendered.
Microcks version or git rev
main @ 4d74e9f
Install method (docker-compose, helm chart, operator, docker-desktop extension,...)
operator
Additional information
Suggested fix: spec.commonExpositions.type.name == "INGRESS", plus a positive and a negative case in MicrocksConfigMapDependentResourceTest. As discussed in #306, I'm happy to send the PR.
AI disclosure: I used an AI assistant to help word this issue; I found the bug while working on #306 and checked it against main.
- Ngôn ngữ chính
- Java
- Star
- 14
- Fork
- 13
- Merge trung bình
- 25 phút
- Pull request đã merge (30 ngày)
- 1
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
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 microcks/microcks-operator
-
post-release.sh: push to main on microcks.ioCó thể đã có người làm @yada đã nhận 3 ngày trước. Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 86/100
microcks/microcks-operator#334 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 64/100
microcks/microcks-operator#332 · 1 bình luận ·
Tất cả issue của microcks/microcks-operator
Issue tương tự
-
Document and enforce pull request review guidelinesCó thể đã có người làm @dashorst đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
Maintainer thường phản hồi trong vòng 1 ngày
-
`Time.at(t, in: nil)` returns UTC instead of the local time (or the time zone of `t`), unlike CRubyĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
hermes-queue
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 72/100
chrisalunlloyd2-sudo/mindpalace#349 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Retry-After parser accepts invalid delay-seconds valuesCó thể đã có người làm @Dhruvraj-Rana đã nhận hôm nay. Đang mởBug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
open-telemetry/opentelemetry-java#9018 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
dropwizard-testing: DAOTest.inTransaction leaves the transaction open when an Error is thrownĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
dropwizard/dropwizard#11410 ·
Maintainer thường phản hồi trong vòng 1 ngày