Skip to content
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

Cmd / Ctrl + clicking a URL with %2Fs causes the %2Fs to be replaced with slashes #68763

Closed
tallpants opened this issue Feb 15, 2019 · 3 comments
Assignees
Labels
*duplicate Issue identified as a duplicate of another issue(s) editor-contrib Editor collection of extras

Comments

@tallpants
Copy link

Not sure why this is happening, but could be related to #4315.

Example:
screenshot 2019-02-15 at 2 37 00 pm

On cmd + clicking this is the URL that gets opened:

screenshot 2019-02-15 at 2 38 43 pm

Confirmed that this happens with every browser (at least on macOS). Currently running version 1.31.11

@vscodebot vscodebot bot added the editor-contrib Editor collection of extras label Feb 15, 2019
@tallpants
Copy link
Author

I was able to narrow it down to here -- https://github.com/Microsoft/vscode/blob/master/src/vs/base/common/uri.ts#L267

For reference, I came across this issue while trying to cmd + click an AWS S3 presigned URL (that have %2F in the path).

The example URL in the image above is invalid in Chromium -- but here's a valid URL that window.open opens just fine in Chromium: https://something.com/hello%2Fworld

But in vscode, what gets passed to window.open (here: https://github.com/Microsoft/vscode/blob/master/src/vs/base/browser/dom.ts#L1151) is already modified and no longer matches the original URL.

@kieferrm
Copy link
Member

/duplicate of #25852

@vscodebot vscodebot bot added the *duplicate Issue identified as a duplicate of another issue(s) label Feb 15, 2019
@vscodebot
Copy link

vscodebot bot commented Feb 15, 2019

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for existing issues here. See also our issue reporting guidelines.

Happy Coding!

@vscodebot vscodebot bot closed this as completed Feb 15, 2019
@vscodebot vscodebot bot locked and limited conversation to collaborators Apr 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*duplicate Issue identified as a duplicate of another issue(s) editor-contrib Editor collection of extras
Projects
None yet
Development

No branches or pull requests

3 participants