Compare commits

...

4 Commits

Author SHA1 Message Date
10d6e90dfe v1.5.15 2023-12-01 17:48:41 +03:00
0183f50cc8 v1.5.14 2023-12-01 12:58:44 +03:00
888923c6ce v1.5.13 2023-11-30 15:01:53 +03:00
e496f8fb7e v1.5.12 2023-11-30 10:55:00 +03:00
3 changed files with 33 additions and 11 deletions

View File

@@ -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

View File

@@ -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)
} }

View File

@@ -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)
} }