Follow-up from "Adding logs to track IT failure exception"
Hello.
The changes from !154 (merged) break integration test steps in CICD.
The following discussions from !154 (merged) should be addressed:
-
@Yan_Sushchynski started a discussion: When
kwargs.get('callback')
returns None, this line of code will fail, because we can't concatenate string andNone
. -
@Yan_Sushchynski started a discussion: There will be the following error here:
TypeError: can only concatenate str (not "dict") to str
I'd recommend using this string-formatting approach:
f"All Header Params: {header_params}"
-
@Yan_Sushchynski started a discussion: Same as here: !154 (comment 124254)
-
@Yan_Sushchynski started a discussion: If there is no
callback
andparams.get('callback')
returnsNone
, this will throwTypeError
.