mirrors/woodpecker
1
0
Fork 0
mirror of https://github.com/woodpecker-ci/woodpecker.git synced 2024-11-20 00:41:02 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #678 from Bugagazavr/docker-client

Browse source 
Return ErrInternalServer when docker server return 500 code
This commit is contained in:
Brad Rydzewski 2014-11-10 00:36:55 -08:00
commit 7354ad8e95
1 changed files with 7 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
shared/build/docker/client.go
View file

@ -162,6 +162,9 @@ var (
// Returned if the specified resource does not exist. // Returned if the specified resource does not exist.
ErrNotFound = errors.New("Not Found") ErrNotFound = errors.New("Not Found")
// Return if something going wrong
ErrInternalServer = errors.New("Internal Server Error")
// Returned if the caller attempts to make a call or modify a resource // Returned if the caller attempts to make a call or modify a resource
// for which the caller is not authorized. // for which the caller is not authorized.
// //
@ -219,6 +222,8 @@ func (c *Client) do(method, path string, in, out interface{}) error {
// Check for an http error status (ie not 200 StatusOK) // Check for an http error status (ie not 200 StatusOK)
switch resp.StatusCode { switch resp.StatusCode {
case 500:
return ErrInternalServer
case 404: case 404:
return ErrNotFound return ErrNotFound
case 403: case 403:
@ -318,6 +323,8 @@ func (c *Client) stream(method, path string, in io.Reader, out io.Writer, header
// Check for an http error status (ie not 200 StatusOK) // Check for an http error status (ie not 200 StatusOK)
switch resp.StatusCode { switch resp.StatusCode {
case 500:
return ErrInternalServer
case 404: case 404:
return ErrNotFound return ErrNotFound
case 403: case 403: