Update LegitimateURLShortener.txt

[Nojuuu]

Remove sa parameter. Adguard's list already removes it from /search path, but I added more paths that I think should be ok, and also limited it to just the document to play it safe.

Example urls:

• https://www.google.com/flights?q=test&hl=fi&source=lnms&tbm=flm&sa=X
• https://www.google.com/finance?sa=X
• https://www.google.com/search?q=test&source=lnms&sa=X
• https://maps.google.com/maps?q=new+york&source=lmns&entry=mt&hl=en&sa=X
• https://www.google.com/webhp?sa=X

Something like ||google.$document,removeparam=/^sa=X$/ could also work, but I don't have time to test everywhere, the only problem I know with removing this parameter is that if you remove it from /url? path but that path also has issues if you remove other params like ved.

---

Maybe old documentation, but might give some idea of what it is for:

https://developers.google.com/analytics/devguides/collection/protocol/v1/parameters#sa

[DandelionSprout]

I'm tempted to add something like simply ||google.$removeparam=/^sa=X$/, especially since I see that the parameter's removal doesn't seem to spell any immediate problems with image searches. I'll see when I have time to handle that.

[iam-py-test]

It does have other values, i.e. https://www.google.com/search?q=actually+legitimate+url+shortener+tool&tbm=vid&ei=zGVqZLfGCu2aptQPjdOV2AM&start=10&sa=N&ved=2ahUKEwi33Lr-h4f_AhVtjYkEHY1pBTsQ8tMDegQIExAE&biw=1280&bih=675&dpr=1.5

[Nojuuu]

If /url? has issues with any parameter removed maybe it needs @@||google.*/url?$removeparam ?

[Nojuuu]

@iam-py-test Where did you find sa=N? Also links in /finance "Today's financial news" have /url?sa=j and removing that causes that redirect error too so if there is /url?sa=X links hidden somewhere, people will probably complain about breakages, but I couldn't find any.

[iam-py-test]

I think from clicking on videos

[DandelionSprout]

I got sa=N when I clicked to get to page 2 or 3 or such on Google text search results.

[Nojuuu]

I got sa=N when I clicked to get to page 2 or 3 or such on Google text search results.

Ok, now I see it too.

[Nojuuu]

@DandelionSprout I suppose we could also try adding filter for sa=N, as there have been no reports of problems with sa=X in the first week?

[DandelionSprout]

I suppose so.