CorsPatchFilter NPE

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

added 1 deleted label

marked the checklist item All of the methods are commented to expectation as completed

marked the checklist item There are no unnecessary files present in the MR as completed

marked the checklist item The continuous integration has no problems with the MR as completed

marked the checklist item The continuous integration has no problems with the MR as incomplete

marked the checklist item The MR is filled in as requested (including labels, milestones, and reviewers) as completed

marked the checklist item The documentation is up-to-date as completed

marked the checklist item All nullable parameters are marked as such as completed

approved this merge request

marked the checklist item The continuous integration has no problems with the MR as completed

marked the checklist item The MR is filled in as requested (including labels, milestones, and reviewers) as incomplete

Please create issue for this MR.

mentioned in issue #242 (closed)

changed the description

marked the checklist item The MR is filled in as requested (including labels, milestones, and reviewers) as completed

unapproved this merge request

approved this merge request

resolved all threads

unapproved this merge request

changed the description

resolved all threads

Could you make a regression test for this?

closed

reopened

approved this merge request

approved this merge request

merged

mentioned in commit 17ebcc13

unassigned @eruighaver