Bladeren bron

Don't error out on plugin err with json

We don't want to error out when there is a json unmarshal error since
the `old way` will cause this to error.

Signed-off-by: Brian Goff <cpuguy83@gmail.com>
Brian Goff 9 jaren geleden
bovenliggende
commit
279dd092b6
1 gewijzigde bestanden met toevoegingen van 4 en 5 verwijderingen
  1. 4 5
      pkg/plugins/client.go

+ 4 - 5
pkg/plugins/client.go

@@ -134,11 +134,10 @@ func (c *Client) callWithRetry(serviceMethod string, data io.Reader, retry bool)
 				Err string
 			}
 			remoteErr := responseErr{}
-			if err := json.Unmarshal(b, &remoteErr); err != nil {
-				return nil, fmt.Errorf("%s: %s", serviceMethod, err)
-			}
-			if remoteErr.Err != "" {
-				return nil, fmt.Errorf("%s: %s", serviceMethod, remoteErr.Err)
+			if err := json.Unmarshal(b, &remoteErr); err == nil {
+				if remoteErr.Err != "" {
+					return nil, fmt.Errorf("%s: %s", serviceMethod, remoteErr.Err)
+				}
 			}
 			// old way...
 			return nil, fmt.Errorf("%s: %s", serviceMethod, string(b))