Merge pull request #214 from Bpazy/master

figo · web-flow · commit b39080e20f18 · 2021-02-26T10:41:24.000-08:00
Jenkins.Init() should check Response.StatusCode #210
diff --git a/jenkins.go b/jenkins.go
@@ -61,7 +61,7 @@ func (j *Jenkins) Init(ctx context.Context) (*Jenkins, error) {
 	}
 
 	j.Version = rsp.Header.Get("X-Jenkins")
-	if j.Raw == nil {
+	if j.Raw == nil || rsp.StatusCode != http.StatusOK {
 		return nil, errors.New("Connection Failed, Please verify that the host and credentials are correct.")
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -61,7 +61,7 @@ func (j Jenkins) Init(ctx context.Context) (Jenkins, error) {`
`61`	`61`	`}`
`62`	`62`
`63`	`63`	`j.Version = rsp.Header.Get("X-Jenkins")`
`64`		`- if j.Raw == nil {`
	`64`	`+ if j.Raw == nil \|\| rsp.StatusCode != http.StatusOK {`
`65`	`65`	`return nil, errors.New("Connection Failed, Please verify that the host and credentials are correct.")`
`66`	`66`	`}`
`67`	`67`