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.

Sign up for GitHub

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

Jump to bottom

file uri not decoded by wslview #295

Closed
1 task done
champignoom opened this issue Nov 9, 2023 · 1 comment
Closed
1 task done

file uri not decoded by wslview #295

champignoom opened this issue Nov 9, 2023 · 1 comment
Assignees
@patrick330602
Labels
Bug This is either an issue that describes a bug or a PR that resolves a bug Status: Patched This is fixed and is part of a released version. Target: 4.x This is targeted to the 4.x release.

Comments

@champignoom
Copy link

Windows Version

Windows 10

Windows Build Number

19045.3570

WSL Version

WSL 2

Distro Version

Ubuntu 22.04

WSL Utilities Version

wslu v3.2.3-1; wslview v10

Describe the bug

escape sequence like %20 in a file uri is not properly decoded by wslview.

Steps to Reproduce

wslview 'file:///home/me/a%20b.txt', then a%20b.txt is opened.

Expected behavior

a b.txt should be opened instead.

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@champignoom champignoom added the Bug This is either an issue that describes a bug or a PR that resolves a bug label Nov 9, 2023
@champignoom champignoom mentioned this issue Nov 9, 2023
Merged
@patrick330602 patrick330602 added Status: In Progress This is being worked on and not ready. Target: 4.x This is targeted to the 4.x release. labels Dec 29, 2023
patrick330602 added a commit that referenced this issue Dec 29, 2023
@patrick330602
fix: [#295] decode file protocol url properly
7c6a9ed
@patrick330602 patrick330602 added Status: Pending Release This is fixed but not yet released. and removed Status: In Progress This is being worked on and not ready. labels Dec 30, 2023
@patrick330602 patrick330602 mentioned this issue Dec 31, 2023
Merged
@PhyX-Meow
Copy link

Same issue here.

@patrick330602 patrick330602 added Status: Patched This is fixed and is part of a released version. and removed Status: Pending Release This is fixed but not yet released. labels Mar 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@patrick330602 patrick330602

Labels
Bug This is either an issue that describes a bug or a PR that resolves a bug Status: Patched This is fixed and is part of a released version. Target: 4.x This is targeted to the 4.x release.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@patrick330602 @PhyX-Meow @champignoom