Skip to content

fix: Resolve GitHub API service initialization and rate limiting issues#70

Merged
Einswilli merged 1 commit intomasterfrom
feature/detailed-error-logging
Mar 30, 2026
Merged

fix: Resolve GitHub API service initialization and rate limiting issues#70
Einswilli merged 1 commit intomasterfrom
feature/detailed-error-logging

Conversation

@Einswilli
Copy link
Copy Markdown
Contributor

  • Initialize missing attributes in GitHubAPIService (_lock, _rate_limit_info, callbacks)
  • Fix datetime import to use from datetime import datetime
  • Configure HTTP client and Github client in init method
  • Remove redundant HTTP client re-initialization in update_rate_limit_info
  • Fix _get_tag method to be synchronous (called from asyncio.to_thread)
  • Change get_repository_info to use HTTP client instead of PyGithub sync client
  • Fix duplicate callback resetting in RateLimiter.set_rate_limit_callback
  • Increase default delays in RateLimiter to be more conservative
  • Use HTTP client for repository info requests to avoid PyGithub threading issues

These changes resolve the 'GitHubAPIService object has no attribute github_client' error and fix rate limit handling issues that were causing 403 errors even with authentication.

- Initialize missing attributes in GitHubAPIService (_lock, _rate_limit_info, callbacks)
- Fix datetime import to use from datetime import datetime
- Configure HTTP client and Github client in __init__ method
- Remove redundant HTTP client re-initialization in update_rate_limit_info
- Fix _get_tag method to be synchronous (called from asyncio.to_thread)
- Change get_repository_info to use HTTP client instead of PyGithub sync client
- Fix duplicate callback resetting in RateLimiter.set_rate_limit_callback
- Increase default delays in RateLimiter to be more conservative
- Use HTTP client for repository info requests to avoid PyGithub threading issues

These changes resolve the 'GitHubAPIService object has no attribute github_client' error and fix rate limit handling issues that were causing 403 errors even with authentication.
@Einswilli Einswilli merged commit 6843cd0 into master Mar 30, 2026
2 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.

2 participants