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

Fix/solving 500 errors #2161

Open
wants to merge 2 commits into
base: release/6.0
Choose a base branch
from
Open

Fix/solving 500 errors #2161

wants to merge 2 commits into from

Conversation

RidhamShah
Copy link
Collaborator

@RidhamShah RidhamShah commented Dec 12, 2024 •
edited
Loading

Cheery-pick some commits to throw perfect 404 User not found errors instead of throwing 500.

@RidhamShah RidhamShah self-assigned this Dec 12, 2024
@RidhamShah RidhamShah linked an issue Dec 12, 2024 that may be closed by this pull request
some errors in prod v5/useraliases, v5/log 500 instead of 404. #2134
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@VivekFitkariwala VivekFitkariwala Awaiting requested review from VivekFitkariwala

@ppratikcr7 ppratikcr7 Awaiting requested review from ppratikcr7

@danoswaltCL danoswaltCL Awaiting requested review from danoswaltCL

At least 1 approving review is required to merge this pull request.

Assignees

@RidhamShah RidhamShah

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

some errors in prod v5/useraliases, v5/log 500 instead of 404.
2 participants
@RidhamShah @ppratikcr7