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.

Sign up for GitHub

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

Jump to bottom

404 instead of 500 error on bad groupID or userID #149

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

404 instead of 500 error on bad groupID or userID #149

wants to merge 1 commit into from

Conversation

abaevbog
Copy link
Contributor

Applies to endpoints like /groups/BAD_NAME/items or /users/BAD_NAME/items

Fixes: #96

@abaevbog
404 instead of 500 error on bad groupID or userID
37b90d6 
Applies to endpoints like /groups/BAD_NAME/items or /users/BAD_NAME/items

Fixes: zotero#96
@abaevbog abaevbog requested a review from dstillman June 14, 2023 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dstillman dstillman Awaiting requested review from dstillman

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Return 400 (or 404?) for invalid group id
1 participant
@abaevbog