Merge pull request #17282 from mtrmac/PodRmReport

Log data that we failed to unmarshal
containers · Jan 31, 2023 · ddd8a61 · ddd8a61
2 parents 56b9476 + 7c60a78
commit ddd8a61
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/pkg/bindings/errors.go b/pkg/bindings/errors.go
@@ -15,7 +15,7 @@ var (
 
 func handleError(data []byte, unmarshalErrorInto interface{}) error {
 	if err := json.Unmarshal(data, unmarshalErrorInto); err != nil {
-		return err
+		return fmt.Errorf("unmarshaling error into %#v, data %q: %w", unmarshalErrorInto, string(data), err)
 	}
 	return unmarshalErrorInto.(error)
 }
@@ -35,7 +35,10 @@ func (h APIResponse) ProcessWithError(unmarshalInto interface{}, unmarshalErrorI
 	}
 	if h.IsSuccess() || h.IsRedirection() {
 		if unmarshalInto != nil {
-			return json.Unmarshal(data, unmarshalInto)
+			if err := json.Unmarshal(data, unmarshalInto); err != nil {
+				return fmt.Errorf("unmarshaling into %#v, data %q: %w", unmarshalInto, string(data), err)
+			}
+			return nil
 		}
 		return nil
 	}