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

Update bad header content response #29474

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@younesdiouri

younesdiouri commented Dec 5, 2018

Q A
Branch? master
Bug fix? No
New feature? No
BC breaks? no
Deprecations? no
Tests pass? no
Fixed tickets
License MIT
Doc PR

In the previous version the dispatcher continues it's road and the output error is not reflecting it's true origin. Hence I tried to add an exception regarding this matter (I am no master in PHP yet ^^ ) .

Update bad header content response
In the previous version the dispatcher continues it's road and the output error is not reflecting it's true origin. Hence I added an exception regarding this matter.

@chalasr chalasr added the Security label Dec 9, 2018

@chalasr chalasr added this to the next milestone Dec 9, 2018

@chalasr

This comment has been minimized.

Member

chalasr commented Dec 9, 2018

Closing as explained, thank you for proposing.

@chalasr chalasr closed this Dec 9, 2018

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