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

Allow configuring showCause in ErrorHandler #11587

Merged
merged 18 commits into from
Mar 29, 2024

Conversation

sbordet
Copy link
Contributor

@sbordet sbordet commented Mar 28, 2024

Penetration testing tools seems to be sensitive to the word "Exception".

ErrorHandler has now an option to configure whether to show the cause in the generated error page.

dhs3000 and others added 16 commits March 22, 2024 08:39
* Issue #11548 - Replication as jetty-ee10-webapp testcase
* Correct order of WebAppContext.stop

---------

Co-authored-by: Jan Bartel <[email protected]>
Co-authored-by: gregw <[email protected]>
…11576)

* Issue #11572 add check for isDeployable when hot deploy file changes
…11575)

* Issue #11574 war should not be extracted unless configured to do so
… to existing destination files. (#11540)

* Issue #11539 restore old behavior of Resource.copyTo()
* Do not delete destination resource
  if it exists before copy to restore
  the behavior to what we had in
  Jetty 9/10/11.
* Formalizing IO.resolvePath()
…ervlet specs for illegal & suspicious characters (#11496)

* Issue #11495 - Add UriCompliance rules that follow the HTTP / URI / Servlet specs for illegal & suspicious characters
* more illegalPathCharacterData test cases
* Correcting SUSPICIOUS_PATH_CHARACTERS semantic (encoded and decoded)
* Check for illegal and suspicious characters as we are parsing the path.
* Only look for ambiguous paths if we know there are dots or encodings.

---------
Co-authored-by: gregw <[email protected]>
…ng, remove obsolete parameter (#11544)

Signed-off-by: Olivier Lamy <[email protected]>
Initialized PEM directories for both client and server.
Added [lib] section that was missing due to #11263.

Signed-off-by: Simone Bordet <[email protected]>
Code cleanups.

Signed-off-by: Simone Bordet <[email protected]>
@sbordet
Copy link
Contributor Author

sbordet commented Mar 28, 2024

Continuation of #11552.

@joakime
Copy link
Contributor

joakime commented Mar 28, 2024

Why are there 43 files changed in this PR??

@joakime
Copy link
Contributor

joakime commented Mar 28, 2024

Try a merge from origin/jetty-12.0.x to see if things improve.

joakime
joakime previously approved these changes Mar 28, 2024
@sbordet sbordet merged commit 1b05d49 into jetty-12.0.x Mar 29, 2024
7 of 10 checks passed
@sbordet sbordet deleted the dhs3000-allow_not_show_causes_in_error_pages branch March 29, 2024 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

5 participants