mirror of
https://github.com/containers/podman.git
synced 2025-05-23 01:57:56 +08:00

* Add response.Body.Close() where needed to release HTTP connections to API server. * Add tests to ensure no general leaks occur. 100% coverage would be required to ensure no leaks on any call. * Update code comments to be godoc correct Signed-off-by: Jhon Honce <jhonce@redhat.com>
47 lines
1.1 KiB
Go
47 lines
1.1 KiB
Go
package bindings
|
|
|
|
import (
|
|
"encoding/json"
|
|
"io/ioutil"
|
|
|
|
"github.com/containers/podman/v3/pkg/errorhandling"
|
|
"github.com/pkg/errors"
|
|
)
|
|
|
|
var (
|
|
ErrNotImplemented = errors.New("function not implemented")
|
|
)
|
|
|
|
func handleError(data []byte) error {
|
|
e := errorhandling.ErrorModel{}
|
|
if err := json.Unmarshal(data, &e); err != nil {
|
|
return err
|
|
}
|
|
return e
|
|
}
|
|
|
|
// Process drains the response body, and processes the HTTP status code
|
|
// Note: Closing the response.Body is left to the caller
|
|
func (h APIResponse) Process(unmarshalInto interface{}) error {
|
|
data, err := ioutil.ReadAll(h.Response.Body)
|
|
if err != nil {
|
|
return errors.Wrap(err, "unable to process API response")
|
|
}
|
|
if h.IsSuccess() || h.IsRedirection() {
|
|
if unmarshalInto != nil {
|
|
return json.Unmarshal(data, unmarshalInto)
|
|
}
|
|
return nil
|
|
}
|
|
// TODO should we add a debug here with the response code?
|
|
return handleError(data)
|
|
}
|
|
|
|
func CheckResponseCode(inError error) (int, error) {
|
|
e, ok := inError.(errorhandling.ErrorModel)
|
|
if !ok {
|
|
return -1, errors.New("error is not type ErrorModel")
|
|
}
|
|
return e.Code(), nil
|
|
}
|