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

[2.8.x] Introduce DevHttpErrorHandler #11305

Merged
merged 10 commits into from Jun 2, 2022

Conversation

mkurz
Copy link
Member

@mkurz mkurz commented Jun 1, 2022

No description provided.

@mkurz mkurz added this to the 2.8.16 milestone Jun 1, 2022
@gmethvin gmethvin merged commit 296784b into playframework:2.8.x Jun 2, 2022
@mkurz mkurz deleted the DevHttpErrorHandler branch June 2, 2022 05:42
@mkurz
Copy link
Member Author

mkurz commented May 9, 2023

Backports of 710885e (GHSA) and #11306

TylerHorth pushed a commit to linkedin/playframework that referenced this pull request Apr 22, 2024
Change obj refs to DefaultHttpErrorHandler to class refs

Introduced a DevErrorHandler, and made the DefaultHttpErrorHandler not leak dev errors

Fixing whitespace/compile errors

Removing stray change from first commit

Formatting

Preserving Server.scala behavior from before

Changing Server.getHandlerFor to take a parameterized fallback error handler

Update dependencies

Use DevHttpErrorHandler to check if play.editor link gets rendered

Revert dependency
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

Successfully merging this pull request may close these issues.

None yet

3 participants