Fixed #1266 @restpath failing on query string or extra path info

[cognitivegears]

This PR proposes a fix for #1266.

Make sure that you've checked the boxes below before you submit PR:

• [ x ] My code includes positive and negative tests.
• [ x ] I have an appropriate description with correct grammar.
• [ x ] I have read Contribution guidelines, maintainers note and Quality standard.
• [ x ] My code is properly linted and passes pre-commit tests.

Thanks for your contribution ❤️

[fzipi]

I wonder why no pipelines have run here 🤔

[cognitivegears]

I wonder why no pipelines have run here 🤔

It's because I'm a first time contributor to this project- the pipelines require approval from a maintainer in order to run.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.76%. Comparing base (07571c6) to head (7f29504).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1267      +/-   ##
==========================================
+ Coverage   81.69%   81.76%   +0.06%     
==========================================
  Files         169      169              
  Lines        9770     9770              
==========================================
+ Hits         7982     7988       +6     
+ Misses       1537     1533       -4     
+ Partials      251      249       -2     

Flag	Coverage Δ
coraza.rule.case_sensitive_args_keys	81.71% <100.00%> (+0.06%)	⬆️
coraza.rule.multiphase_valuation	81.76% <100.00%> (+0.06%)	⬆️
coraza.rule.no_regex_multiline	81.69% <100.00%> (+0.06%)	⬆️
default	81.76% <100.00%> (+0.06%)	⬆️
examples+	16.54% <0.00%> (ø)
examples+coraza.rule.case_sensitive_args_keys	81.71% <100.00%> (+0.06%)	⬆️
examples+coraza.rule.multiphase_valuation	81.59% <100.00%> (+0.06%)	⬆️
examples+coraza.rule.no_regex_multiline	81.61% <100.00%> (+0.06%)	⬆️
examples+memoize_builders	81.71% <100.00%> (+0.06%)	⬆️
examples+no_fs_access	81.04% <100.00%> (+0.06%)	⬆️
ftw	81.76% <100.00%> (+0.06%)	⬆️
memoize_builders	81.85% <100.00%> (+0.06%)	⬆️
no_fs_access	81.20% <100.00%> (+0.06%)	⬆️
tinygo	81.72% <100.00%> (+0.06%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[fzipi]

Looks good to me. There is an extra blank line that we can remove and I think we can merge. I'm trying to think more border cases, but at least there is additional coverage now.

[cognitivegears]

Removed additional blank line.

[fzipi]

LGTM

[M4tteoP]

Thanks, looks good to me also!