Skip to content

Update client error logging#59

Merged
piceri merged 2 commits intomainfrom
update-client-error-logging
Mar 13, 2026
Merged

Update client error logging#59
piceri merged 2 commits intomainfrom
update-client-error-logging

Conversation

@piceri
Copy link
Contributor

@piceri piceri commented Mar 13, 2026

Update deployment record client error logging to improve debugging capabilities.

@piceri piceri marked this pull request as ready for review March 13, 2026 19:42
@piceri piceri requested a review from a team as a code owner March 13, 2026 19:42
Copilot AI review requested due to automatic review settings March 13, 2026 19:42
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates deploymentrecord.Client.PostOne error handling to improve observability when the GitHub deployment records API returns non-2xx responses.

Changes:

  • Moves response body draining/closing into the success branch and adds error-response body reading for logging.
  • Expands client-error (4xx, excluding 429) logs with status_code and response body content.
  • Adds debug logging for retriable non-2xx responses including the response body.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@piceri piceri merged commit 148adab into main Mar 13, 2026
10 checks passed
@piceri piceri deleted the update-client-error-logging branch March 13, 2026 19:58
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.

4 participants