Tweak to rules regex to only match final square brackets

[jgchristian]

If either the source or destination strings contain an opening square bracket, the current Rules regex will match the anything after the first square bracket. Therefore, too much of the rule is considered when determining which action to take

e.g. for the rule /a[z] /bG [L] the Rules regex will match [z] /bG [L]

Then the "Gone" rule will match on the captial G in the destination string and rather than treating this as a standard rewrite, will try to return a 410 status code

[jgchristian]

Hi. That regex will put us back to the current state and the unit test I wrote will fail again

Here's some example urls

• Your regex (matching too much): https://regex101.com/r/zL7eW6/1
• My regex: https://regex101.com/r/aJ2hC8/1

[tinganho]

@jgchristian thanks the non-greedy seem to not work with $.

[tinganho]

Published now on connect-modrewrite@0.8.2. Please try it out.