Compare commits
4 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 10d6e90dfe | |||
| 0183f50cc8 | |||
| 888923c6ce | |||
| e496f8fb7e |
@@ -1,7 +1,4 @@
|
|||||||
## Version 1.5.11
|
## Version 1.5.13
|
||||||
|
|
||||||
### Bugfix
|
### Bugfix
|
||||||
- Fix unhandled error in do() method for client and legacy-client
|
- Fix EOF error by closing request
|
||||||
|
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
20
client.go
20
client.go
@@ -140,15 +140,25 @@ func (dc *DecortClient) do(req *http.Request) (*http.Response, error) {
|
|||||||
req.Header.Add("Authorization", "bearer "+dc.cfg.Token)
|
req.Header.Add("Authorization", "bearer "+dc.cfg.Token)
|
||||||
req.Header.Set("Accept", "application/json")
|
req.Header.Set("Accept", "application/json")
|
||||||
|
|
||||||
// var resp *http.Response
|
|
||||||
// var err error
|
|
||||||
buf, _ := io.ReadAll(req.Body)
|
buf, _ := io.ReadAll(req.Body)
|
||||||
|
|
||||||
// for i := uint64(0); i < dc.cfg.Retries; i++ {
|
|
||||||
// req = req.Clone(req.Context())
|
|
||||||
req.Body = io.NopCloser(bytes.NewBuffer(buf))
|
req.Body = io.NopCloser(bytes.NewBuffer(buf))
|
||||||
resp, err := dc.client.Do(req)
|
resp, err := dc.client.Do(req)
|
||||||
|
req.Close = true
|
||||||
if err != nil || resp == nil {
|
if err != nil || resp == nil {
|
||||||
|
if strings.Contains(err.Error(), "connection reset by peer") || errors.Is(err, io.EOF) {
|
||||||
|
for i := uint64(0); i < dc.cfg.Retries; i++ {
|
||||||
|
time.Sleep(5 * time.Second)
|
||||||
|
resp, err = dc.client.Do(req)
|
||||||
|
if strings.Contains(err.Error(), "connection reset by peer") || errors.Is(err, io.EOF) {
|
||||||
|
continue
|
||||||
|
} else if err == nil {
|
||||||
|
break
|
||||||
|
} else if err != nil {
|
||||||
|
return nil, err
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
return resp, err
|
return resp, err
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -159,6 +169,6 @@ func (dc *DecortClient) do(req *http.Request) (*http.Response, error) {
|
|||||||
respBytes, _ := io.ReadAll(resp.Body)
|
respBytes, _ := io.ReadAll(resp.Body)
|
||||||
err = fmt.Errorf("%s", respBytes)
|
err = fmt.Errorf("%s", respBytes)
|
||||||
resp.Body.Close()
|
resp.Body.Close()
|
||||||
|
|
||||||
return nil, fmt.Errorf("could not execute request: %w", err)
|
return nil, fmt.Errorf("could not execute request: %w", err)
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -145,7 +145,21 @@ func (ldc *LegacyDecortClient) do(req *http.Request) (*http.Response, error) {
|
|||||||
// req = req.Clone(req.Context())
|
// req = req.Clone(req.Context())
|
||||||
req.Body = io.NopCloser(bytes.NewBuffer(buf))
|
req.Body = io.NopCloser(bytes.NewBuffer(buf))
|
||||||
resp, err := ldc.client.Do(req)
|
resp, err := ldc.client.Do(req)
|
||||||
|
req.Close = true
|
||||||
if err != nil || resp == nil {
|
if err != nil || resp == nil {
|
||||||
|
if strings.Contains(err.Error(), "connection reset by peer") || errors.Is(err, io.EOF) {
|
||||||
|
for i := uint64(0); i < ldc.cfg.Retries; i++ {
|
||||||
|
time.Sleep(5 * time.Second)
|
||||||
|
resp, err = ldc.client.Do(req)
|
||||||
|
if strings.Contains(err.Error(), "connection reset by peer") || errors.Is(err, io.EOF) {
|
||||||
|
continue
|
||||||
|
} else if err == nil {
|
||||||
|
break
|
||||||
|
} else if err != nil {
|
||||||
|
return nil, err
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
return resp, err
|
return resp, err
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -156,6 +170,7 @@ func (ldc *LegacyDecortClient) do(req *http.Request) (*http.Response, error) {
|
|||||||
respBytes, _ := io.ReadAll(resp.Body)
|
respBytes, _ := io.ReadAll(resp.Body)
|
||||||
err = fmt.Errorf("%s", respBytes)
|
err = fmt.Errorf("%s", respBytes)
|
||||||
resp.Body.Close()
|
resp.Body.Close()
|
||||||
|
// }
|
||||||
|
|
||||||
return nil, fmt.Errorf("could not execute request: %w", err)
|
return nil, fmt.Errorf("could not execute request: %w", err)
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user