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

DSP-777 Show message on all server errors in login component #200

Merged
merged 2 commits into from
Oct 5, 2020

Conversation

kilchenmann
Copy link
Contributor

resolves DSP-777

@kilchenmann kilchenmann self-assigned this Oct 5, 2020
@kilchenmann kilchenmann added the enhancement New feature or request label Oct 5, 2020
@mdelez
Copy link
Contributor

mdelez commented Oct 5, 2020

can you give me a short explanation on how to test this please?

@kilchenmann
Copy link
Contributor Author

@mdelez Sure. You can turn of the database for example: Stop the corresponding docker container. You can use the docker dashboard for this or the command line with docker stop CONTAINER_ID. The api should still run and will give you internal server error (status 500)

@kilchenmann
Copy link
Contributor Author

kilchenmann commented Oct 5, 2020

When knora-stack is not running (no api, no db, no sipi, no salsah) you get a status 0

Screenshot 2020-10-05 at 11 23 36

@kilchenmann
Copy link
Contributor Author

In case of a broken knora-stack (maybe only the api is running), you get a 503 error:

Screenshot 2020-10-05 at 11 27 26

Copy link
Contributor

@mdelez mdelez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just making the English a bit clearer

@kilchenmann kilchenmann merged commit 4aac16b into master Oct 5, 2020
@kilchenmann kilchenmann deleted the wip/dsp-777-login-server-error branch October 5, 2020 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants