Logger always highlights first match

[joey04]

The first match is almost always the actual match, but sometimes it's not.

I saw it again here with this plain string rule that's word-boundary enforced by default:

(Another case for me is #3011, but that requires the change I made to enforce word-boundary for > 7 char tokens.)

---

update:

I wrote this addition, which finds the actual match for every case I've come across.

(I've since refined my patch to be a do-while(1) loop with a single null test, still using the exact same regex statements. Functionally, this just saves a few CPU cycles, but it's a nice aesthetic improvement.)

[joey04]

Closing per #3011 (comment)

[gorhill]

I thinks its best that the logger shows the real match which occurred inside uBO.

[joey04]

I agree, but are there any other cases besides the prefix1's I reported? If not, then it's a niche consideration like #3011.

[gorhill]

I couldn't reproduce with the original URL provided, however I could reproduce and verify the fix using those hand-crafted URL/filter pairs:

• http://example.org/1s.crazyclickstats.com/1/? with /1
• http://example.org/123456s.crazyclickstats.com/123456/? with /123456
• http://example.org/1234567s.crazyclickstats.com/1234567/? with /1234567