HGI- 10444: handle REQUEST_LIMIT_EXCEEDED 403 error as InvalidCredentialsError#221
Open
raheelhotglue wants to merge 111 commits into
Open
HGI- 10444: handle REQUEST_LIMIT_EXCEEDED 403 error as InvalidCredentialsError#221raheelhotglue wants to merge 111 commits into
raheelhotglue wants to merge 111 commits into
Conversation
raheelhotglue
commented
Jun 17, 2026
raheelhotglue
commented
- Added REQUEST_LIMIT_EXCEEDED to the 403 error handling
Fix/memory
fix duplicates
remove validade json as default
added listviews states
fix state error
added Reports
disable reports by default
fix missing sobjects in listView
Report list and target csv name fix
--------- Co-authored-by: Massab <massab@MacBook-Pro.local>
--------- Co-authored-by: LiorB-D <liorbd@outlook.com> Co-authored-by: Hassan Syyid <hassansyyid@gmail.com>
* feature/hgi-8473 * refactor: improve combined query logic --------- Co-authored-by: massab <massab@massabs-MacBook-Pro.local>
…54) Co-authored-by: massab <massab@massabs-MacBook-Pro.local>
…#56) - Added backoff mechanism for handling connection failures during API calls. - Refactored _process_response_chunks to handle invalid chunk lengths and other exceptions gracefully. - Updated exception handling to raise RetriableError for specific issues.
- Updated sync_stream to return counter value directly. - Introduced _chunk_list and _execute_chunked_query functions to handle large record IDs in chunks. - Enhanced sync_filtered_accounts to utilize chunked queries for better performance and reliability.
HGi-8814: Add chunked query execution for large record sets
… Unit Tests (#65) * HGI-9665: Add initial test suite, CI configuration and pre-commit for Unit Tests * Update README to clarify pre-commit hook installation for tests before commits and pushes * HGI-9665: Address CursorBot feedback
* HGI-9863: Setup automated linter * Cleanup
* HGI-9899: migrate to HotglueSingerSDK cli driver * HGI-9899: cursor fix * HGI-9899: lint fix * HGI-9899: cursor fix * HGI-9899: preserve catalog metadata * Add alerting --------- Co-authored-by: LiorB-D <liorbd@outlook.com>
…Auth failures (#69) - Import InvalidCredentialsError from hotglue_singer_sdk.tap_base - Change exception raised in login() method to InvalidCredentialsError - This ensures the monitoring stack can properly identify credential errors without triggering false alerts Resolves HGI-9951 Co-authored-by: Cursor Agent <cursoragent@cursor.com>
* HGI-9948: Support access token retrieval via HG API * Fix: Remove redundant condition for auth_endpoint in SalesforceTap class * Clean up * Enhance sandbox detection logic in SalesforceTap class to handle string values for is_sandbox configuration
fix(HG-6156): add dummy commit
…n to report_ids (#75)
- Added REQUEST_LIMIT_EXCEEDED to the 403 error handling
… raising TapSalesforceQuotaExceededException instead of InvalidCredentialsError.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
16 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.