Decoding protobufs when a new field comes in breaks
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 42/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- ruby
- Lĩnh vực
- backend, distributed-systems
Hướng nghiên cứu
Bắt đầu tại dòng 18 của lib/temporal/connection/converter/payload/proto_json.rb và kiểm tra cách trình phân tích protobuf của Ruby xử lý các trường được một dịch vụ Go upstream thêm vào. Tái hiện ParseError với một trường không xác định và xác định xem thư viện hay Google protobufs kiểm soát hành vi này. Hoàn tất khi một worker Ruby Temporal có thể deserialize thông báo mà không thất bại do trường mới.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
There's a bug when an upstream service adds a new protocol buffer field and sends it along to a Ruby temporal worker. For example, we have a service written in Go that has an update protocol buffer message that is being serialized. The activity that executes with the message as an input is written in Ruby. The error is:
Google::Protobuf::ParseError: Error occurred during parsing: Error parsing JSON @1:845: No such field: newField
The offending line is here: https://github.com/coinbase/temporal-ruby/blob/3fbc675fbe24bce236fba2376910fd5dc9a9ff5f/lib/temporal/connection/converter/payload/proto_json.rb#L18
Honestly, I don't know if this is the responsibility of this library or from Google protobufs. In Go, you can explicitly say ignore unknown fields, however Ruby does not seem to have this behavior. So I'm not sure what to do or if anyone has any exceptions. Protobufs should be compatible with deserializing into messages it doesn't know about new fields.
- Ngôn ngữ chính
- Ruby
- Star
- 287
- Fork
- 113
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
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 coinbase/temporal-ruby
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
coinbase/temporal-ruby#361 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
coinbase/temporal-ruby#341 ·
-
Emitting Metrics for PrometheusĐang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
coinbase/temporal-ruby#328 · 1 bình luận ·
-
NoMethodError when await_workflow_result is called for long running workflowsCó thể đã có người làm @iekundayo đã nhận 712 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
coinbase/temporal-ruby#326 · 1 bình luận ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
coinbase/temporal-ruby#324 · 3 bình luận · 2 reaction ·
Tất cả issue của coinbase/temporal-ruby
Issue tương tự
-
agent:triaged bug bughunt pm:bundler priority:p1
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
SocketDev/socket-patch#1573 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug chat verified
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 6 ngày
-
🐞 bug pending triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
CircuitVerse/CircuitVerse#8031 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Empty members in the Accept header cause a 500Có thể đã có người làm @koic đã 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
modelcontextprotocol/ruby-sdk#609 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
BazaRb::Fake#enter rejects dotted badges accepted by BazaRb#enterCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 1/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