Skip to content

[PB-5908]: add global retry with rate-limit notifications for SDK clients#1866

Open
terrerox wants to merge 7 commits intomasterfrom
feature/global-retry
Open

[PB-5908]: add global retry with rate-limit notifications for SDK clients#1866
terrerox wants to merge 7 commits intomasterfrom
feature/global-retry

Conversation

@terrerox
Copy link
Contributor

Description

Upgrade @internxt/sdk to 1.15.0 and introduce retry strategies for HTTP requests. Silent retries (2 attempts) are enabled globally, while Storage and Share clients use a user-facing notification strategy (5 attempts) with a cooldown-based warning toast. Adds hasElapsed utility to date.service and i18n translations for the rate-limit toast in all supported languages

Related Issues

Related Pull Requests

Checklist

  • Changes have been tested locally.
  • Unit tests have been written or updated as necessary.
  • The code adheres to the repository's coding standards.
  • Relevant documentation has been added or updated.
  • No new warnings or errors have been introduced.
  • SonarCloud issues have been reviewed and addressed.
  • QA Passed

Testing Process

Additional Notes

Upgrade @internxt/sdk to 1.15.0 and introduce retry strategies for HTTP requests. Silent retries (2 attempts) are enabled globally, while Storage and Share clients use a user-facing notification strategy (5 attempts) with a cooldown-based warning toast. Adds hasElapsed utility to date.service and i18n translations for the rate-limit toast in all supported languages
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Feb 24, 2026

Deploying drive-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: 5886a11
Status: ✅  Deploy successful!
Preview URL: https://8d716c46.drive-web.pages.dev
Branch Preview URL: https://feature-global-retry.drive-web.pages.dev

View logs

- uses: actions/setup-node@v4
with:
node-version: '20.11.1'
node-version: '20.19.0'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Bumped from 20.11.1 to 20.19.0 to fix failing Playwright CI job.
@noble/hashes@2.0.1 (transitive dep of @internxt/sdk@1.15.0) requires Node >= 20.19.0,
which caused yarn install to fail with "Found incompatible module".

@terrerox terrerox self-assigned this Feb 25, 2026
@terrerox terrerox requested a review from CandelR February 27, 2026 03:59
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 2, 2026

@terrerox terrerox requested a review from CandelR March 3, 2026 15:35
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