fix: omit response body from config push error (#284)

Omit the response body from config push error strings.
ReloadDeclarativeRawConfig already returns the body separately, so
formatting it into the error is unnecessary.

CHANGELOG.md

@@ -1,5 +1,6 @@
 # Table of Contents
 
+- [v0.38.1](#v0381)
 - [v0.38.0](#v0380)
 - [v0.37.0](#v0370)
 - [v0.36.0](#v0360)
@@ -49,6 +50,15 @@
 
 ## [v0.38.0]
 
+> Release date: 2023/02/22
+
+- Omit response body from error when config push fails. The body can be quite
+  large and will result in massive logs downstream if included in the error.
+  The body is returned along with the error and downstream clients and log it
+  separately if they so desire.
+
+## [v0.38.0]
+
 > Release date: 2023/02/13
 
 - Added `instance_name` to Plugin schema.
@@ -692,6 +702,7 @@ authentication credentials in Kong.
   releases of Kong since every release of Kong is introducing breaking changes
   to the Admin API.
 
+[v0.38.1]: https://github.com/Kong/go-kong/compare/v0.38.0...v0.38.1
 [v0.38.0]: https://github.com/Kong/go-kong/compare/v0.37.0...v0.38.0
 [v0.37.0]: https://github.com/Kong/go-kong/compare/v0.36.0...v0.37.0
 [v0.36.0]: https://github.com/Kong/go-kong/compare/v0.35.0...v0.36.0

kong/client.go

@@ -421,7 +421,7 @@ func (c *Client) ReloadDeclarativeRawConfig(
 		return nil, fmt.Errorf("could not read /config %d status response body: %w", resp.StatusCode, err)
 	}
 	if resp.StatusCode < 200 || resp.StatusCode >= 400 {
-		return b, fmt.Errorf("failed posting new config to /config: got status code %d and body %s", resp.StatusCode, b)
+		return b, fmt.Errorf("failed posting new config to /config: got status code %d", resp.StatusCode)
 	}
 
 	return b, nil