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

Check returned status codes in various integrations and cases #2270

Open
patrick91 opened this issue Oct 21, 2022 · 1 comment
Open

Check returned status codes in various integrations and cases #2270

patrick91 opened this issue Oct 21, 2022 · 1 comment

Comments

@patrick91
Copy link
Member

patrick91 commented Oct 21, 2022

in #1809 and #2266 I've changed the views to all have the same response codes for all cases, but I'm not 100% sure the status code make a lot of sense for all cases.

I think we can use this issue to discuss what the view should return on the various cases, for example when doing a GET to /graphql with graphiql disable return 404, but it returns 200 when sending a valid query and we allow queries via GET.

This should be done once we merge #1809

Upvote & Fund

  • We're using Polar.sh so you can upvote and help fund this issue.
  • We receive the funding once the issue is completed & confirmed by you.
  • Thank you in advance for helping prioritize & fund our backlog.
Fund with Polar
@jkimbo
Copy link
Member

jkimbo commented Oct 24, 2022

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

No branches or pull requests

2 participants