Skip to content

[2.x] fix: Log server response body on publish failure#8537

Merged
eed3si9n merged 2 commits intosbt:developfrom
MkDev11:fix/log-publish-error-response-7423
Jan 15, 2026
Merged

[2.x] fix: Log server response body on publish failure#8537
eed3si9n merged 2 commits intosbt:developfrom
MkDev11:fix/log-publish-error-response-7423

Conversation

@MkDev11
Copy link
Copy Markdown
Contributor

@MkDev11 MkDev11 commented Jan 15, 2026

When publishing to a repository fails with an HTTP error (e.g., 403, 409), the server often includes helpful error details in the response body. Previously, sbt only showed the HTTP status code without the response body.

This adds ErrorLoggingURLHandler which extends BasicURLHandler and logs the error stream content when an upload fails, making it easier to diagnose publishing issues.

Fixes #7423


Contribution by Gittensor, see my contribution statistics at https://gittensor.io/miners/details?githubId=94194147

@MkDev11
Copy link
Copy Markdown
Contributor Author

MkDev11 commented Jan 15, 2026

@eed3si9n please have a look at the PR when you're available

@MkDev11 MkDev11 force-pushed the fix/log-publish-error-response-7423 branch from 0f8f384 to a303f22 Compare January 15, 2026 04:30
Copy link
Copy Markdown
Member

@eed3si9n eed3si9n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@eed3si9n eed3si9n changed the title fix: Log server response body on publish failure [2.x] fix: Log server response body on publish failure Jan 15, 2026
@eed3si9n eed3si9n merged commit b2db557 into sbt:develop Jan 15, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Log server's response payload in case of publishing failure

2 participants