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

Add status error codes to responses #66

Open
ckniffen opened this issue Dec 19, 2022 · 2 comments
Open

Add status error codes to responses #66

ckniffen opened this issue Dec 19, 2022 · 2 comments

Comments

@ckniffen
Copy link
Collaborator

Error responses from the VHS always return a status of 200. We should leverage 404 and 500s at the very least.

404 example https://github.com/ripple/validator-history-service/blob/main/src/api/routes/v1/validator.ts#L233
500 example https://github.com/ripple/validator-history-service/blob/main/src/api/routes/v1/validator.ts#L275

@sohammk08
Copy link

Is this issue still valid? And if yes, should I work on it?

@mvadari
Copy link
Collaborator

mvadari commented Oct 19, 2023

I believe so, and sure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants