New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(logging): Support numbers when unmarshaling Severity #4216
fix(logging): Support numbers when unmarshaling Severity #4216
Conversation
logging.Entry.Severity is written out as a JSON number during Marshalling but has an UnmarshalJSON method that only supports Severity as as string. This commit adds support to unmarshal JSON numbers in addition to strings so Marshalling then Unmarshalling the same object works without error
Thanks for this! |
No region tags are edited in this PR.This comment is generated by snippet-bot.
|
@alutman do you want us to merge this - is it ready? With how CI is configured and given this repo has a lot of PRs updating changes, you mgiht be stuck in a cycle of having to update this branch, and then wait for someone internal to approve HEAD, in order for CI to run, before it can be merged Let us know if this PR is ready, and I can merge for you. |
@nicoleczhu |
logging.Entry.Severity is written out as a JSON number during
Marshalling but has an UnmarshalJSON method that only supports Severity
as as string.
This commit adds support to unmarshal JSON numbers in addition to
strings so Marshalling then Unmarshalling the same object works without
error
Fixes #4215