You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@devlake.apache.org by GitBox <gi...@apache.org> on 2022/09/03 02:13:11 UTC

[GitHub] [incubator-devlake] e2corporation opened a new issue, #2940: [Bug][Core] Database Migration Required uses Wrong Status Code 500

e2corporation opened a new issue, #2940:
URL: https://github.com/apache/incubator-devlake/issues/2940

   ### Search before asking
   
   - [X] I had searched in the [issues](https://github.com/apache/incubator-devlake/issues?q=is%3Aissue) and found no similar issues.
   
   
   ### What happened
   
   As part of the assumptions for a required DevLake Database Migration, the Backend should return with a HTTP Status Code `428`, instead it using Internal Server `500`.
   
   ### What you expected to happen
   
   API should always return with Status Code `428` If DB Migration is Required
   
   ### How to reproduce
   
   N/A - UI will not show migration alert if api responds with internal server error 500
   
   ### Anything else
   
   _No response_
   
   ### Version
   
   main
   
   ### Are you willing to submit PR?
   
   - [ ] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-devlake] klesh commented on issue #2940: [Bug][Core] Database Migration Required uses Wrong Status Code 500

Posted by GitBox <gi...@apache.org>.
klesh commented on issue #2940:
URL: https://github.com/apache/incubator-devlake/issues/2940#issuecomment-1236028234

   @keon94 I think this might be related to the PR #2763 
   Would you like to take a look?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-devlake] klesh closed issue #2940: [Bug][Core] Database Migration Required uses Wrong Status Code 500

Posted by GitBox <gi...@apache.org>.
klesh closed issue #2940: [Bug][Core] Database Migration Required uses Wrong Status Code 500 
URL: https://github.com/apache/incubator-devlake/issues/2940


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-devlake] keon94 commented on issue #2940: [Bug][Core] Database Migration Required uses Wrong Status Code 500

Posted by GitBox <gi...@apache.org>.
keon94 commented on issue #2940:
URL: https://github.com/apache/incubator-devlake/issues/2940#issuecomment-1236056428

   > @keon94 I think this might be related to the PR #2763 Would you like to take a look?
   
   Yeah, it's a minor issue. I can fix it.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org