sindresorhus/refined-github

Do you want to work on this issue?

You can request for a bounty in order to promote it!

Branches with multiple slashes aren't fully supported #4492

KSH-code posted onGitHub

<!-- Thanks for reporting a bug! ⛰ Make sure the bug is caused by Refined GitHub. Try disabling the extension first. Show us the bug and help us replicate it, include: 1. A REAL URL where the bug appears. If it happens on a private repo, find an equivalent public URL. 2. A screenshot/video/gif of the issue, if it’s visual. 3. Any related errors in the browser’s console, if any. -->

  1. https://github.com/yakov116/TestR/tree/this/branch%2Fhas%2Fmany%2Fslashes
  2. <img width="1051" alt="스크린샷 2021-06-18 오후 6 15 15" src="https://user-images.githubusercontent.com/29705162/122538129-35537880-d061-11eb-93be-9eca84a9d91f.png">

  1. <img width="541" alt="스크린샷 2021-06-18 오후 6 13 51" src="https://user-images.githubusercontent.com/29705162/122537862-f4f3fa80-d060-11eb-86f3-ccc421090dc4.png">

The problem found from https://github.com/sindresorhus/refined-github/pull/4474 this is a task afterwards https://github.com/sindresorhus/refined-github/pull/4474


If the path has %2F, it's already not ambiguous and should not be passed to disambiguateReference

a/b%2Fc is unambiguously the name of the branch. Names won't have %2F in the filename (because it'd be represented as a/b%252Fc in the URL) and / cannot be part of the filename.

Originally posted by @fregante in https://github.com/sindresorhus/refined-github/pull/4474#discussion_r652610916

posted by fregante almost 4 years ago

Fund this Issue

$0.00
Funded
Only logged in users can fund an issue

Pull requests