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

Provide error pages #658

Open
snicoll opened this issue Mar 12, 2021 · 6 comments
Open

Provide error pages #658

snicoll opened this issue Mar 12, 2021 · 6 comments

Comments

@snicoll
Copy link
Contributor

snicoll commented Mar 12, 2021

Now that we've re-enabled the whitelabel page, we should provide a static error page that's a bit more inline with the current UI.

@10xtechie
Copy link

@snicoll ,
Do you mean that we should have a static error page e.g error.html and fill it with appropriate HTTP error code such as 404, 500 etc - with a generic error massage? Also, do you want any further details about the error; for example: which java exception causes the 500 error, so on and so forth?

@snicoll
Copy link
Contributor Author

snicoll commented Jun 16, 2022

@10xtechie thanks for the interest. I am not sure how we're going to approach this to be honest. Our web UI is doing things that may impact what we decide to do. Essentially, this is about providing a proper error page when something goes wrong (with the same UI styling). But it is also about making sure that if you hit the service directly, you get back the usual error attributes.

@10xtechie
Copy link

10xtechie commented Jun 18, 2022

@snicoll
Thanks so very much for your reply!!
Let me think aloud here: on the sever side, I would create a generic error handling controller which will produce usual error attributes both in HTML and JSON and on the client side, a generic error page template, which is consistent with our web UI and will consume data produced by that generic error controller. Please let me know what you think :)

@snicoll
Copy link
Contributor Author

snicoll commented Jun 19, 2022

I would create a generic error handling controller

That is supposed to be taken care by Spring Boot's default error handling. Unfortunately this proves to be a bit tricky with our web client for some reason. I am happy for you giving a try. We can discuss based on a PR.

@10xtechie
Copy link

@snicoll ,
Thanks for insights! I will try and create a PR. Please assign it to me :)

10xtechie added a commit to 10xtechie/start.spring.io that referenced this issue Jul 7, 2022
@10xtechie
Copy link

@snicoll ,
Thanks for assigning this issue to me! I have created a PR for the same. Please have a look.

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

No branches or pull requests

2 participants