Escape Subversion URLs

 






 

 


  Review Request #361
  —
  Created April 17, 2008 and submitted

 


















 

  

   Information

  





 
 

  djs52
 









 
 

  Review Board SVN (deprecated)
 









 
 

  
 









 
 

  
 









 
 

  
 






 




 

  

   Reviewers

  





 
 

  reviewboard
 









 
 

  
 






 








 





  


 
 

  
This patch escapes the URL passed to Subversion so that paths with spaces in them work correctly. Without this, a cryptic "Malformed repository URL" message is shown to the user when submitting via the new review request form.

 








  


 
 

  
Verified working with patches to files with spaces in the filenames and spaces in the paths, for both http:// and file:/// repository URLs.

 







 









 





  








 









 






  

 








 














 





 

  

   




david




   

   





  

 



 

  


   

   

    
 
david



    
   


  

  

  



    




  1. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    2. 

















  2. 
 
    
  
  
    
  
 
    
  
    
   
    
    
     
    
      
       trunk/reviewboard/scmtools/svn.py
       


        (Diff revision 1)


       
      
     
    
        
    
     
     
    

     
     
    

     
     
    

     
     
    

     
     
    

     
     
    

     
     
    

     
     
    

     
     
    

        
   
    
  
    
 
    

   
    
    
    





    
    Also, it seems like __normalize_path is a natural place for this to go, instead of here.
    
  
    
 
    

 
    
  


    
 
    
 
      

  


    1. 
 
 
      
 
      
  
 
      
 
      
  
      Well, yes and no. __normalize_path seems to be about figuring out whether the path is relative to the repo or already includes the repo. Escaping seems like a different task. I'm happy to do it either way though.
      
 
      

      2. 



    2. 
 
 
      
 
      
  
 
      
 
      
  
      OK, I guess that makes sense.  It's fine to leave this here.
      
 
      

      3. 


 
    


    

 
    

    3. 






  3. 
 
    
  
  
    
  
 
    
  
    
   
    
    
     
    
      
       trunk/reviewboard/scmtools/svn.py
       


        (Diff revision 1)


       
      
     
    
        
    
     
     
    

     
     
    

     
     
    

     
     
    

     
     
    

     
     
    

        
   
    
  
    
 
    

   
    
    
    





    
    Can you add a comment explaining why this is here?
    
  
    
 
    

 
    
  


    
 
    
 
      

  


    1. 
 
 
      
 
      
  
 
      
 
      
  
      Do you mean the urlsplit or the whole block of code? (the urlsplit is there to avoid escaping the : in http://)
      
 
      

      2. 



    2. 
 
 
      
 
      
  
 
      
 
      
  
      I mean the whole block.
      
 
      

      3. 



    3. 
 
 
      
 
      
  
 
      
 
      
  
      See updated patch
      
 
      

      4. 


 
    


    

 
    

    4. 




  4. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    5. 




  

 
















 





 

  

   




david




   

   







  

 



 

  


   

   

    
 
david



    
   


  

  

  



    




  1. 
 
    
  
    
   
    Committed as r1310.  Thanks!
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    2. 















  2. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    3.