-
Notifications
You must be signed in to change notification settings - Fork 853
TypeError - decode() argument 'encoding' must be str, not None #829
Copy link
Copy link
Closed
Labels
Version: 2xbugM-T: A confirmed bug report. Issues are confirmed when the reproduction steps are documentedM-T: A confirmed bug report. Issues are confirmed when the reproduction steps are documentedweb-client
Milestone
Description
It seems slack just had a blip and returned some kind of error responses that did not feature the charset header. This broke in the SDK with:
TypeError - decode() argument 'encoding' must be str, not None
On this line:
From the Sentry local-caputring I can see charset was None - in typing terms it's Optional[str] being used as a str.
Reproducible in:
Requires slack returning these unexpected responses, or simulating them
The Slack SDK version
2.9.1
Python runtime version
3.8.5
OS info
Linux
Steps to reproduce:
As above
Expected result:
Some kind of fallback decoding?
Actual result:
TypeError raised
Requirements
For general questions/issues about Slack API platform or its server-side, could you submit questions at https://my.slack.com/help/requests/new instead. 🙇
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Version: 2xbugM-T: A confirmed bug report. Issues are confirmed when the reproduction steps are documentedM-T: A confirmed bug report. Issues are confirmed when the reproduction steps are documentedweb-client