Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Raise ForbiddenError when a 403 is encountered #59

Closed
rconradharris opened this issue Oct 27, 2014 · 1 comment
Closed

Raise ForbiddenError when a 403 is encountered #59

rconradharris opened this issue Oct 27, 2014 · 1 comment
Assignees
Labels

Comments

@rconradharris
Copy link
Contributor

@rconradharris rconradharris commented Oct 27, 2014

If you a user doesn't have rights to a particular error, the code will currently raise an UnknownError with 403 embedded in the message.

To make this easier to handle, we should add a new exception ForbiddenError which the caller can use to catch handle appropriately.

rconradharris added a commit to rconradharris/python-redmine that referenced this issue Oct 27, 2014
Replaces the `UnknownError` that was previously raised.

Fixes maxtepkeev#59
rconradharris added a commit to rconradharris/python-redmine that referenced this issue Oct 27, 2014
Replaces the `UnknownError` that was previously raised.

Fixes maxtepkeev#59
@maxtepkeev maxtepkeev self-assigned this Oct 27, 2014
@maxtepkeev
Copy link
Owner

@maxtepkeev maxtepkeev commented Oct 27, 2014

Thanks! That makes perfect sense!

maxtepkeev added a commit that referenced this issue Oct 27, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

2 participants
You can’t perform that action at this time.