Feature: Add __str__ to OAuthRefreshException#107
Conversation
|
Warning Review limit reached
More reviews will be available in 50 minutes and 4 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
Adds a __str__ method to OAuthRefreshException so stringifying the exception produces a more descriptive message that includes the error and error_description fields from the OAuth response when available, falling back to the inherited message.
Changes:
- Implement
__str__onOAuthRefreshExceptionwith three formatting branches (both fields, error only, message only).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| def __str__(self): | ||
| if self.error and self.error_description: | ||
| return f'{self.message}: {self.error} - {self.error_description}' | ||
|
|
||
| if self.error: | ||
| return f'{self.message}: {self.error}' | ||
|
|
||
| return self.message |
| def __str__(self): | ||
| if self.error and self.error_description: | ||
| return f'{self.message}: {self.error} - {self.error_description}' | ||
|
|
||
| if self.error: | ||
| return f'{self.message}: {self.error}' | ||
|
|
||
| return self.message |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Extracted from #104