GODRIVER-831: don't pass empty payload when beginning GSSAPI negotiation.

craiggwilson · craiggwilson · commit 4934d52c45fe · 2019-02-15T06:43:58.000-06:00
diff --git a/x/mongo/driver/auth/internal/gssapi/gss.go b/x/mongo/driver/auth/internal/gssapi/gss.go
@@ -94,7 +94,9 @@ func (sc *SaslClient) Start() (string, []byte, error) {
 		return mechName, nil, sc.getError("unable to initialize client")
 	}
 
-	return mechName, nil, nil
+	payload, err := sc.Next(nil)
+
+	return mechName, payload, err
 }
 
 func (sc *SaslClient) Next(challenge []byte) ([]byte, error) {
diff --git a/x/mongo/driver/auth/internal/gssapi/sspi.go b/x/mongo/driver/auth/internal/gssapi/sspi.go
@@ -109,7 +109,9 @@ func (sc *SaslClient) Start() (string, []byte, error) {
 		return mechName, nil, sc.getError("unable to intitialize client")
 	}
 
-	return mechName, nil, nil
+	payload, err := sc.Next(nil)
+
+	return mechName, payload, err
 }
 
 func (sc *SaslClient) Next(challenge []byte) ([]byte, error) {

Original file line number	Diff line number	Diff line change
`@@ -94,7 +94,9 @@ func (sc *SaslClient) Start() (string, []byte, error) {`
`94`	`94`	`return mechName, nil, sc.getError("unable to initialize client")`
`95`	`95`	`}`
`96`	`96`
`97`		`- return mechName, nil, nil`
	`97`	`+ payload, err := sc.Next(nil)`
	`98`	`+`
	`99`	`+ return mechName, payload, err`
`98`	`100`	`}`
`99`	`101`
`100`	`102`	`func (sc *SaslClient) Next(challenge []byte) ([]byte, error) {`
Original file line number	Diff line number	Diff line change
`@@ -109,7 +109,9 @@ func (sc *SaslClient) Start() (string, []byte, error) {`
`109`	`109`	`return mechName, nil, sc.getError("unable to intitialize client")`
`110`	`110`	`}`
`111`	`111`
`112`		`- return mechName, nil, nil`
	`112`	`+ payload, err := sc.Next(nil)`
	`113`	`+`
	`114`	`+ return mechName, payload, err`
`113`	`115`	`}`
`114`	`116`
`115`	`117`	`func (sc *SaslClient) Next(challenge []byte) ([]byte, error) {`