Querystring filter

[shaunhurley]

Tom,

Working optional querystring filter code as discussed in issue #23

Added a new field to the redirect definition called ignoreQUerystring and the rest of it is pretty much in line with your suggestions.

Cheers,

Shaun

[boutell]

This looks great, thank you. A couple minor notes, and I'd appreciate an entry in the Changelog section of README.md.

If you're too busy let me know though, these are good changes and I don't mind doing cleanup if needed.

[shaunhurley]

Code updated and changes pushed, should be all good.

[boutell]

Thanks for the contribution! This should come out in our next release cycle (2 weeks).

[abea]

Is there new functionality here that should be covered in the README?

[shaunhurley]

Oops, that's embarrassing - I didn't push up the README edit. Tom, do you want/need me to do a new pull request for that?

"By default a redirect includes any query string on incoming requests when matching for redirection. You can toggle the 'Ignore query string when matching.' option to ignore query strings on requests and only match on the base URL path."

[boutell]

D'oh, I missed it too! Would you mind Shaun? Thanks.

…

On Wed, Feb 10, 2021 at 12:08 PM Shaun Hurley ***@***.***> wrote: Oops, that's embarrassing - I didn't push up the README edit. Tom, do you want/need me to do a new pull request for that? "By default a redirect includes any query string on incoming requests when matching for redirection. You can toggle the 'Ignore query string when matching.' option to ignore query strings on requests and only match on the base URL path." — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#24 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAH27OTTXN5UVRH4JBQO6TS6K4QLANCNFSM4XL5G3BA> .

-- THOMAS BOUTELL | CHIEF TECHNOLOGY OFFICER APOSTROPHECMS | apostrophecms.com | he/him/his