Update onvif error checking to use explict None check (#92642)
parent
4a2af45e37
commit
c624e50b60
|
@ -17,16 +17,16 @@ def stringify_onvif_error(error: Exception) -> str:
|
||||||
"""Stringify ONVIF error."""
|
"""Stringify ONVIF error."""
|
||||||
if isinstance(error, Fault):
|
if isinstance(error, Fault):
|
||||||
message = error.message
|
message = error.message
|
||||||
if error.detail:
|
if error.detail is not None: # checking true is deprecated
|
||||||
# Detail may be a bytes object, so we need to convert it to string
|
# Detail may be a bytes object, so we need to convert it to string
|
||||||
if isinstance(error.detail, bytes):
|
if isinstance(error.detail, bytes):
|
||||||
detail = error.detail.decode("utf-8", "replace")
|
detail = error.detail.decode("utf-8", "replace")
|
||||||
else:
|
else:
|
||||||
detail = str(error.detail)
|
detail = str(error.detail)
|
||||||
message += ": " + detail
|
message += ": " + detail
|
||||||
if error.code:
|
if error.code is not None: # checking true is deprecated
|
||||||
message += f" (code:{error.code})"
|
message += f" (code:{error.code})"
|
||||||
if error.subcodes:
|
if error.subcodes is not None: # checking true is deprecated
|
||||||
message += (
|
message += (
|
||||||
f" (subcodes:{','.join(extract_subcodes_as_strings(error.subcodes))})"
|
f" (subcodes:{','.join(extract_subcodes_as_strings(error.subcodes))})"
|
||||||
)
|
)
|
||||||
|
|
Loading…
Reference in New Issue