[x] Added tests for code changes or test/build only changes
[x] Updated the change log file (CHANGES.md|CHANGELOG.md) or test/build only changes
[x] Completed the PR template below:
Description
Added encoding='utf-8' to responses to avoid slow chardet path.
Fixes #414
Approach
Check if the repsonse encoding is unset and if so set it to utf-8.
(Unrelated) Removed unnecessarypass statements causing lint failures.
Schema & API Changes
"No change"
Security and Privacy
"No change"
Testing
No new tests because this path is covered by existing tests. We don't have any performance tests where we could automatically validate that this fix works or check for regressions.
Checklist
CHANGES.md
|CHANGELOG.md
) or test/build only changesDescription
Added
encoding='utf-8'
to responses to avoid slowchardet
path.Fixes #414
Approach
Check if the repsonse encoding is unset and if so set it to
utf-8
. (Unrelated) Removed unnecessarypass
statements causing lint failures.Schema & API Changes
Security and Privacy
Testing
Monitoring and Logging