这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@rakeshkky
Copy link
Member

Description

The server used to throw HTTP 500 exception for invalid _like pattern. Change it to
400 Bad Request.

Affected components

  • Server
  • Tests

Related Issues

fix #2486

Solution and Design

Capture 22025 Postgres error code and convert it into 400 Bad Request.

Steps to test and verify

Follow the reproduction guide in #2486

Limitations, known bugs & workarounds

@rakeshkky rakeshkky added s/ok-to-merge Status: This pull request can be merged to master c/server Related to server e/quickfix can be wrapped up in few hours labels Aug 5, 2019
@rakeshkky rakeshkky requested a review from lexi-lambda as a code owner August 5, 2019 11:34
@rakeshkky rakeshkky self-assigned this Aug 5, 2019
@netlify
Copy link

netlify bot commented Aug 5, 2019

Deploy preview for hasura-docs ready!

Built with commit c70d92b

https://deploy-preview-2671--hasura-docs.netlify.com

@hasura-bot
Copy link
Contributor

Review app for commit f305a93 deployed to Heroku: https://hge-ci-pull-2671.herokuapp.com
Docker image for server: hasura/graphql-engine:pull2671-f305a93b

@rakeshkky rakeshkky requested a review from ecthiender August 8, 2019 06:52
lexi-lambda
lexi-lambda previously approved these changes Aug 8, 2019
@hasura-bot
Copy link
Contributor

Review app for commit c70d92b deployed to Heroku: https://hge-ci-pull-2671.herokuapp.com
Docker image for server: hasura/graphql-engine:pull2671-c70d92b0

@lexi-lambda lexi-lambda merged commit a03c788 into hasura:master Aug 8, 2019
@hasura-bot
Copy link
Contributor

Review app https://hge-ci-pull-2671.herokuapp.com is deleted

@dsandip dsandip added this to the beta.5 milestone Aug 19, 2019
polRk pushed a commit to polRk/graphql-engine that referenced this pull request Feb 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c/server Related to server e/quickfix can be wrapped up in few hours s/ok-to-merge Status: This pull request can be merged to master

Projects

None yet

Development

Successfully merging this pull request may close these issues.

capture Postgres error code 22025 as HTTP 400 bad request

5 participants