CorsPatchFilter NPE
All threads resolved!
All threads resolved!
Description
Fixes a bug where the CorsPatchFilter would throw an NPE on a request without an origin header.
Resolves #242 (closed)
Changes
CorsPatchFilter::requiresHeaderPatchCheck
now returns false on requests without a header.
Test and Review
To be filled in by the reviewers
-
All of the methods are commented to expectation -
The methods are tested to satisfaction -
There are no unnecessary files present in the MR -
The continuous integration has no problems with the MR -
The MR is filled in as requested (including labels, milestones, and reviewers) -
The documentation is up-to-date -
All nullable parameters are marked as such
Edited by Erik Oudsen
Merge request reports
Activity
- Resolved by Ewoud Ruighaver
Please create issue for this MR.
mentioned in issue #242 (closed)
mentioned in commit 17ebcc13
unassigned @eruighaver
Please register or sign in to reply