Skip to content

Commit

Permalink
feat: add logic to convert severity string to uppercase (#744)
Browse files Browse the repository at this point in the history
  • Loading branch information
ymotongpoo committed Apr 21, 2023
1 parent 7baafa0 commit c1c8ce1
Show file tree
Hide file tree
Showing 3 changed files with 97 additions and 71 deletions.
5 changes: 4 additions & 1 deletion google/cloud/logging_v2/entries.py
Original file line number Diff line number Diff line change
Expand Up @@ -229,7 +229,10 @@ def to_api_repr(self):
if self.insert_id is not None:
info["insertId"] = self.insert_id
if self.severity is not None:
info["severity"] = self.severity
if isinstance(self.severity, str):
info["severity"] = self.severity.upper()
else:
info["severity"] = self.severity
if self.http_request is not None:
info["httpRequest"] = self.http_request
if self.timestamp is not None:
Expand Down
2 changes: 1 addition & 1 deletion google/cloud/logging_v2/logger.py
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,7 @@ def _do_log(self, client, _entry_class, payload=None, **kw):
kw["resource"] = kw.pop("resource", self.default_resource)

severity = kw.get("severity", None)
if isinstance(severity, str) and not severity.isupper():
if isinstance(severity, str):
# convert severity to upper case, as expected by enum definition
kw["severity"] = severity.upper()

Expand Down

0 comments on commit c1c8ce1

Please sign in to comment.