code/twython
1
Fork 0
Code Issues 23 Pull requests 4 Projects Releases 5 Packages Wiki Activity Actions

TwythonErrors should display errors with the built-in Exception args, not its own msg #171

New issue
Closed
opened 2013-04-11 16:28:21 -07:00 by nofeet · 0 comments
nofeet commented 2013-04-11 16:28:21 -07:00 (Migrated from github.com)
Copy link

Sentry, our team's error logging platform, does not prominently display the error message whenever a TwythonError is raised. I believe it is happening because Twython's Exception classes set and use self.msg but Sentry is calling repl() which returns self.args.

It would be better if the TwythonErrors used the parent Exception's attributes and str() method.

More information about this at http://www.python.org/dev/peps/pep-0352/.

Sentry, our team's error logging platform, does not prominently display the error message whenever a `TwythonError` is raised. I believe it is happening because Twython's Exception classes set and use `self.msg` but Sentry is calling `repl()` which returns `self.args`. It would be better if the TwythonErrors used the parent Exception's attributes and str() method. More information about this at http://www.python.org/dev/peps/pep-0352/.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
master
bugs/371
gh-pages
v3.9.1
3.7.0
3.4.0
3.3.0
3.1.2
v3.3.0
Labels
Clear labels   
Bug

   
Enhancement

   
Feature Suggestion
No labels
Bug
Enhancement
Feature Suggestion
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: code/twython#171
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?