Comment by WvBoyScouter on 29/04/2022 at 21:35 UTC

3 upvotes, 1 direct replies (showing 1)

View submission: Amputator has an overactive regex

View parent comment

I wonder if it would be possible to split the URL in to subdomain, domain, and split each of the path folders into an array and regex them separately. At least that might be a place to start if it makes sense to fix it.

Replies

Comment by Killed_Mufasa at 29/04/2022 at 22:56 UTC

3 upvotes, 0 direct replies

Hmm that might work, not a bad idea! That way we wouldn't have to maintain a deny list for certain domains with amp in it, we could just say all urls with the amp string only in the domain are false positives. I imagine it being a bit harder path-wise, but this is for sure something I'll look into. Thx!