Skip to content

The recent addition of linkid parameter stripping breaks all redirects via go.microsoft.com/fwlink/ #46

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
jacobbaratta opened this issue Apr 30, 2021 · 1 comment
Labels

Comments

@jacobbaratta
Copy link

Some generic examples of the link format I'm talking about (with possible additional parameters omitted):

It took me quite a bit of time to figure out that this was the extension at fault for breaking these. The behavior created by this extension is that every one of these links would redirect to bing.com (typically either to https://www.bing.com/?ref=go or to https://www.bing.com/?ref=go&pver=&sbp=&os=0&over=&plcid=&olcid=0&clcid=&ar=&sar=&o1=&o2=&o3=&o4=&o5=) instead of redirecting to where the link is supposed to be redirecting to. As these links are ubiquitous across Microsoft various websites for their products & services, this is a pretty huge problem.

Some actual full real-world examples of these links for reproduction/testing:

In terms of a suggested fix for this issue, you're probably either going to need to carve out an exception for go.microsoft.com, or just remove the linkid stripping entirely.

@jacobbaratta jacobbaratta changed the title Recent addition of linkid parameter stripping breaks all redirects via go.microsoft.com/fwlink/ The recent addition of linkid parameter stripping breaks all redirects via go.microsoft.com/fwlink/ Apr 30, 2021
@jparise jparise added the bug label Apr 30, 2021
@jparise
Copy link
Owner

jparise commented Apr 30, 2021

Thanks for the detailed report, @jacobbaratta. I agree with you, and I've removed linkid stripping (#44) in the upcoming version 2.9 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants