adjust according to feedback

Author: pengux

NEXT_CHANGELOG.md

@@ -12,8 +12,8 @@
 * Added `NewLimitIterator` to `listing` package for lazy iteration with a cap on output items ([#1555](https://github.com/databricks/databricks-sdk-go/pull/1555)).
 
 ### Bug Fixes
-fix: 1243 returns both resetErr and the request's err on retries.Halt
 
+ * Fixed request error being discarded when the request body cannot be reset for retry ([#1363](https://github.com/databricks/databricks-sdk-go/pull/1363)).
  * Fix double-caching of OAuth tokens in Azure client secret credentials ([#1549](https://github.com/databricks/databricks-sdk-go/issues/1549)).
  * Disable async token refresh for GCP credential providers to avoid wasted refresh attempts caused by double-caching with Google's internal `oauth2.ReuseTokenSource` ([#1549](https://github.com/databricks/databricks-sdk-go/issues/1549)).
  * Fixed double-caching in M2M OAuth that prevented the proactive async token refresh from reaching the HTTP endpoint until ~10s before expiry, causing bursts of 401 errors at token rotation boundaries ([#1549](https://github.com/databricks/databricks-sdk-go/issues/1549)).

httpclient/api_client.go

@@ -260,7 +260,7 @@ func (c *ApiClient) handleError(ctx context.Context, err error, body common.Requ
 	}
 	if resetErr := body.Reset(); resetErr != nil {
 		return nil, retries.Halt(
-			fmt.Errorf("%w, request error: %w", resetErr, err),
+			fmt.Errorf("%w (unable to reset request body for retry: %w)", err, resetErr),
 		)
 	}
 	return nil, retries.Continue(err)