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

Weird URLs are converted from '%' to '%25' in Markdown links #156859

Closed
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug markdown Markdown support issues *out-of-scope Posted issue is not in scope of VS Code uri
Milestone

Comments

@kaho2063
Copy link

kaho2063 commented Aug 2, 2022

Issue Type: Bug

In the Markdown source code, when Ctrl + clicking on the URL part of the following description, "%" in the URL is converted to "%25". It doesn't happen when clicking on a link in the preview, or "Follow Link" that pops up.

[Rust Japan Wiki](https://rust-japan.game-info.wiki/d/%A5%C8%A5%C3%A5%D7%A5%DA%A1%BC%A5%B8)
[link-1]: https://rust-japan.game-info.wiki/d/%A5%C8%A5%C3%A5%D7%A5%DA%A1%BC%A5%B8

It does not occur in the URL only description or plain text mode as shown below.

https://rust-japan.game-info.wiki/d/%A5%C8%A5%C3%A5%D7%A5%DA%A1%BC%A5%B8

VS Code version: Code 1.69.2 (3b889b0, 2022-07-18T16:14:10.636Z)
OS version: Linux x64 5.4.0-122-generic
Restricted Mode: No

System Info
Item Value
CPUs Intel(R) Core(TM) i3-5005U CPU @ 2.00GHz (4 x 1817)
GPU Status 2d_canvas: unavailable_software
canvas_oop_rasterization: disabled_off
direct_rendering_display_compositor: disabled_off_ok
gpu_compositing: disabled_software
multiple_raster_threads: enabled_on
opengl: disabled_off
rasterization: disabled_software
raw_draw: disabled_off_ok
skia_renderer: enabled_on
video_decode: disabled_software
video_encode: disabled_software
vulkan: disabled_off
webgl: unavailable_software
webgl2: unavailable_software
Load (avg) 2, 1, 1
Memory (System) 7.69GB (0.35GB free)
Process Argv --unity-launch --crash-reporter-id 05cb9321-6408-47b1-b4d1-ef314d862d3d --crash-reporter-id 05cb9321-6408-47b1-b4d1-ef314d862d3d --crash-reporter-id 05cb9321-6408-47b1-b4d1-ef314d862d3d
Screen Reader no
VM 0%
DESKTOP_SESSION mate
XDG_CURRENT_DESKTOP MATE
XDG_SESSION_DESKTOP mate
XDG_SESSION_TYPE x11
Extensions (8)
Extension Author (truncated) Version
githistory don 0.6.19
EditorConfig Edi 0.16.4
vscode-macros EXC 1.4.0
vscode-language-pack-ja MS- 1.69.7130911
python ms- 2022.10.1
vscode-pylance ms- 2022.7.40
googlesearch nat 0.1.0
markdown-all-in-one yzh 3.4.3
@kaho2063 kaho2063 changed the title "%" converted to "%25" in Markdown links Weird URLs are converted from '%' to '%25' in Markdown links Aug 2, 2022
@mjbvz mjbvz added this to the August 2022 milestone Aug 3, 2022
@mjbvz mjbvz added bug Issue identified by VS Code Team member as probable bug markdown Markdown support issues labels Aug 5, 2022
@kaho2063
Copy link
Author

kaho2063 commented Aug 9, 2022 via email

@meganrogge
Copy link
Contributor

The issue still reproduces for me

@meganrogge meganrogge added the verification-found Issue verification failed label Aug 23, 2022
@meganrogge meganrogge reopened this Aug 23, 2022
@mjbvz mjbvz modified the milestones: August 2022, September 2022 Aug 23, 2022
@mjbvz mjbvz removed the verification-found Issue verification failed label Aug 23, 2022
@mjbvz mjbvz added the author-verification-requested Issues potentially verifiable by issue author label Sep 28, 2022
@rebornix rebornix added verified Verification succeeded verification-found Issue verification failed and removed verified Verification succeeded labels Sep 30, 2022
@rebornix rebornix reopened this Sep 30, 2022
@rebornix
Copy link
Member

I'm still seeing this in today's Insiders.

@mjbvz
Copy link
Collaborator

mjbvz commented Sep 30, 2022

Hm yes only seems fixed on the second link (the definition). Will fix see about full fix in October

@mjbvz mjbvz modified the milestones: September 2022, October 2022 Sep 30, 2022
@mjbvz mjbvz removed verification-found Issue verification failed author-verification-requested Issues potentially verifiable by issue author labels Sep 30, 2022
@mjbvz mjbvz modified the milestones: October 2022, On Deck Sep 30, 2022
@mjbvz
Copy link
Collaborator

mjbvz commented Sep 30, 2022

Actually this is blocked by #85930 afaik. The server is returning the correct uri string, but vscode.Uri.parse mangles the uri. I also tried switching to vscode.Uri.from, but still could not construct the correct uri. Something always ends up being double encoded

Let me know if you have any workarounds @jrieken

@jrieken jrieken added the uri label Sep 30, 2022
@jrieken
Copy link
Member

jrieken commented Sep 30, 2022

Yeah, there is unfortunately no workaround and allowing URL would be the best way going forward

@mjbvz mjbvz added the *out-of-scope Posted issue is not in scope of VS Code label Dec 5, 2022
@vscodenpa
Copy link

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.

If you wonder what we are up to, please see our roadmap and issue reporting guidelines.

Thanks for your understanding, and happy coding!

@vscodenpa vscodenpa closed this as not planned Won't fix, can't repro, duplicate, stale Dec 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue identified by VS Code Team member as probable bug markdown Markdown support issues *out-of-scope Posted issue is not in scope of VS Code uri
Projects
None yet
6 participants