Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: authentication error logging in API client #267

Merged
merged 1 commit into from
Mar 19, 2023
Merged

fix: authentication error logging in API client #267

merged 1 commit into from
Mar 19, 2023

Conversation

bpg
Copy link
Owner

@bpg bpg commented Mar 19, 2023 •
edited
Loading

Community Note

  • Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for pull request followers and do not help prioritize the request

Relates to #265

@bpg bpg force-pushed the logging-api-client-errors branch from 1f3281b to 5d9877e Compare March 19, 2023 01:13
@bpg bpg marked this pull request as ready for review March 19, 2023 01:13
mergify[bot]
mergify bot approved these changes Mar 19, 2023
View reviewed changes
Copy link

@mergify mergify bot left a comment

Choose a reason for hiding this comment

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

🤖 LGTM! beep boop

@bpg bpg enabled auto-merge (squash) March 19, 2023 01:14
@bpg bpg changed the title fix: additional logging in API client chore: additional logging in API client Mar 19, 2023
@bpg bpg merged commit 763527e into main Mar 19, 2023
@bpg bpg deleted the logging-api-client-errors branch March 19, 2023 01:15
@ghost ghost mentioned this pull request Mar 19, 2023
Merged
@bpg bpg changed the title chore: additional logging in API client fix: authentication error logging in API client Mar 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mergify Mergify Mergify approved these changes

Assignees
No one assigned
Labels
autoapprove
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@bpg