-
Notifications
You must be signed in to change notification settings - Fork 45
chore: librarian update image pull request: 20260213T151402Z #668
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -796,7 +796,7 @@ def __call__( | |
| ) | ||
| method = transcoded_request["method"] | ||
| try: | ||
| request_payload = json_format.MessageToJson(request) | ||
| request_payload = type(request).to_json(request) | ||
| except: | ||
| request_payload = None | ||
| http_request = { | ||
|
|
@@ -943,7 +943,7 @@ def __call__( | |
| ) | ||
| method = transcoded_request["method"] | ||
| try: | ||
| request_payload = json_format.MessageToJson(request) | ||
| request_payload = type(request).to_json(request) | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Using a bare |
||
| except: | ||
| request_payload = None | ||
| http_request = { | ||
|
|
@@ -1094,7 +1094,7 @@ def __call__( | |
| ) | ||
| method = transcoded_request["method"] | ||
| try: | ||
| request_payload = json_format.MessageToJson(request) | ||
| request_payload = type(request).to_json(request) | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Using a bare |
||
| except: | ||
| request_payload = None | ||
| http_request = { | ||
|
|
@@ -1395,7 +1395,7 @@ def __call__( | |
| ) | ||
| method = transcoded_request["method"] | ||
| try: | ||
| request_payload = json_format.MessageToJson(request) | ||
| request_payload = type(request).to_json(request) | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Using a bare |
||
| except: | ||
| request_payload = None | ||
| http_request = { | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using a bare
except:on line 800 is discouraged as it catches all exceptions, including system-exiting ones likeKeyboardInterruptandSystemExit. This can hide bugs and make debugging difficult. It's better to catch a more specific exception, likeexcept Exception:, or even more specific exceptions thatto_jsonmight raise.