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

Handle application errors #110

Closed
Tracked by #608
thekaveman opened this issue Aug 3, 2022 · 1 comment · Fixed by #129
Closed
Tracked by #608

Handle application errors #110

thekaveman opened this issue Aug 3, 2022 · 1 comment · Fixed by #129
Assignees
@machikoyasuda
Milestone
Courtesy Cards

Comments

@thekaveman
Copy link
Member

thekaveman commented Aug 3, 2022 •
edited
Loading

See Flask - Handling Application Errors

Specifically, we should look at Returning API Errors as JSON.

  • 403 (forbidden, not authenticated/authorized)
  • 404 (endpoint not found)
  • 500 (other application error / Exception)
@thekaveman thekaveman moved this to Backlog in Digital Services Aug 3, 2022
@thekaveman thekaveman added this to the Courtesy Cards milestone Aug 3, 2022
@machikoyasuda
Copy link
Member

https://flask.palletsprojects.com/en/2.2.x/errorhandling/#returning-api-errors-as-json

@thekaveman thekaveman moved this from Backlog to This Sprint (Month) in Digital Services Aug 4, 2022
@machikoyasuda machikoyasuda self-assigned this Aug 8, 2022
@machikoyasuda machikoyasuda moved this from This Sprint (Month) to In Progress in Digital Services Aug 16, 2022
@machikoyasuda machikoyasuda mentioned this issue Aug 17, 2022
Merged
@thekaveman thekaveman mentioned this issue Aug 24, 2022
Closed
34 tasks
Repository owner moved this from In Progress to Done in Digital Services Aug 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@machikoyasuda machikoyasuda

Labels
None yet
Projects
Digital Services
Archived in project
Milestone
Courtesy Cards
Development

Successfully merging a pull request may close this issue.

Raise JSON errors, add app error handling methods and test cal-itp/eligibility-server
2 participants
@thekaveman @machikoyasuda