issue 295

[joeyzhao2018]

What does this PR do?

• First added a unittest case to replicate the issue
• Add the is_authorizer_response helper method to duck-type the response in a pythonic way.

Motivation

Fix the issue that the str responses were not handled elegantly and causing traceback.print_exc

Testing Guidelines

Added a test case to make sure no matter the response type, no error thrown and no traceback.print_exe in normal case.

Additional Notes

Types of Changes

• Bug fix
• New feature
• Breaking change
• Misc (docs, refactoring, dependency upgrade, etc.)

Check all that apply

• This PR's description is comprehensive
• This PR contains breaking changes that are documented in the description
• This PR introduces new APIs or parameters that are documented and unlikely to change in the foreseeable future
• This PR impacts documentation, and it has been updated (or a ticket has been logged)
• This PR's changes are covered by the automated tests
• This PR collects user input/sensitive content into Datadog
• This PR passes the integration tests (ask a Datadog member to run the tests)

[alexgallotta]

should there be a debug log or something in case DD_ENCODE_AUTHORIZER_CONTEXT is enabled but there is an error preventing the fields to be populated?

[joeyzhao2018]

should there be a debug log or something in case DD_ENCODE_AUTHORIZER_CONTEXT is enabled but there is an error preventing the fields to be populated?

Added the debug log.