plubming: transport, Escape the user and pswd for endpoint. Fixes #723

plumbing/transport/common.go

@@ -128,10 +128,10 @@ func (u *Endpoint) String() string {
 		buf.WriteString("//")
 
 		if u.User != "" || u.Password != "" {
-			buf.WriteString(u.User)
+			buf.WriteString(url.PathEscape(u.User))
 			if u.Password != "" {
 				buf.WriteByte(':')
-				buf.WriteString(u.Password)
+				buf.WriteString(url.PathEscape(u.Password))
 			}
 
 			buf.WriteByte('@')

plumbing/transport/common_test.go

@@ -1,6 +1,7 @@
 package transport
 
 import (
+	"net/url"
 	"testing"
 
 	"gopkg.in/src-d/go-git.v4/plumbing/protocol/packp/capability"
@@ -153,6 +154,15 @@ func (s *SuiteCommon) TestNewEndpointFileURL(c *C) {
 	c.Assert(e.String(), Equals, "file:///foo.git")
 }
 
+func (s *SuiteCommon) TestValidEndpoint(c *C) {
+	e, err := NewEndpoint("http://github.com/user/repository.git")
+	e.User = "[email protected]"
+	e.Password = " !\"#$%&'()*+,-./:;<=>?@[\\]^_`{|}~"
+	url, err := url.Parse(e.String())
+	c.Assert(err, IsNil)
+	c.Assert(url, NotNil)
+}
+
 func (s *SuiteCommon) TestNewEndpointInvalidURL(c *C) {
 	e, err := NewEndpoint("http://\\")
 	c.Assert(err, NotNil)