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

Create a default error page #20

Open
pjsharpe07 opened this issue Aug 12, 2020 · 3 comments
Open

Create a default error page #20

pjsharpe07 opened this issue Aug 12, 2020 · 3 comments

Comments

@pjsharpe07
Copy link

pjsharpe07 commented Aug 12, 2020

Proposed feature

Default error landing page doesn't exist and some tomcat information is sent. Also, the default page exposes sensitive information as well.

See screen shot:

image

Rationale

A default error page could prevent this potentially sensitive information from being inadvertently disclosed.

Additional context

Found this via a nessus scan.

@robertdebock
Copy link
Owner

That is a very good plan.

Do you know how to do this? If so; please create a pull request for it. I'll help you through the process if you need help.

Otherwise, I'll work on it in some time.

Regards,

Robert de Bock.

@robertdebock
Copy link
Owner

Hi @pjsharpe07. The role now places a simple, unbranded default page. That should solve your Nessus finding.

Please let me know if this works for you.

@pjsharpe07
Copy link
Author

Thank you for doing this! Unfortunately, we remove some of those folders so this fix didn't quite work for us.

Instead, we did some work with serverinfo.properties. You can find some of the changes here.

Thanks again for doing this work!

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

No branches or pull requests

2 participants