From f809dde0e719f89a6fb5cb80f3be65f5cbc1d1fe Mon Sep 17 00:00:00 2001
From: Tim Pope <code@tpope.net>
Date: Sun, 26 Jun 2022 17:03:54 -0400
Subject: [PATCH] Use URL encoding for Fugitive URLs

This makes them conform to whatever the RFC for URLs is.
---
 autoload/fugitive.vim | 22 +++++++++++++++-------
 1 file changed, 15 insertions(+), 7 deletions(-)

diff --git a/autoload/fugitive.vim b/autoload/fugitive.vim
index 4646199..29385cd 100644
--- a/autoload/fugitive.vim
+++ b/autoload/fugitive.vim
@@ -88,6 +88,18 @@ function! s:UrlEncode(str) abort
   return substitute(a:str, '[%#?&;+\<> [:cntrl:]]', '\=printf("%%%02X", char2nr(submatch(0)))', 'g')
 endfunction
 
+function! s:PathUrlEncode(str) abort
+  return substitute(a:str, '[%#?[:cntrl:]]', '\=printf("%%%02X", char2nr(submatch(0)))', 'g')
+endfunction
+
+function! s:PathJoin(prefix, str) abort
+  if a:prefix =~# '://'
+    return a:prefix . s:PathUrlEncode(a:str)
+  else
+    return a:prefix . a:str
+  endif
+endfunction
+
 function! s:throw(string) abort
   throw 'fugitive: '.a:string
 endfunction
@@ -553,18 +565,14 @@ endfunction
 if exists('+shellslash')
   function! s:DirUrlPrefix(dir) abort
     let gd = s:GitDir(a:dir)
-    return 'fugitive://' . (gd =~# '^[^/]' ? '/' : '') . gd . '//'
+    return 'fugitive://' . (gd =~# '^[^/]' ? '/' : '') . s:PathUrlEncode(gd) . '//'
   endfunction
 else
   function! s:DirUrlPrefix(dir) abort
-    return 'fugitive://' . s:GitDir(a:dir) . '//'
+    return 'fugitive://' . s:PathUrlEncode(s:GitDir(a:dir)) . '//'
   endfunction
 endif
 
-function! s:PathJoin(prefix, str) abort
-  return a:prefix . a:str
-endfunction
-
 function! s:Tree(...) abort
   return a:0 ? FugitiveWorkTree(a:1) : FugitiveWorkTree()
 endfunction
@@ -1638,7 +1646,7 @@ function! s:DirCommitFile(path) abort
   if empty(vals)
     return ['', '', '']
   endif
-  return [s:Dir(vals[1])] + (empty(vals[2]) ? ['', '/.git/index'] : vals[2:3])
+  return [s:Dir(fugitive#UrlDecode(vals[1])), vals[2], empty(vals[2]) ? '/.git/index' : fugitive#UrlDecode(vals[3])]
 endfunction
 
 function! s:DirRev(url) abort