Skip to content

feat: maintain reference to refresh-promise to avoid repetitive calls#7

Merged
rickygarg merged 7 commits intomainfrom
retain-refresh-promise-reference
Oct 22, 2024
Merged

feat: maintain reference to refresh-promise to avoid repetitive calls#7
rickygarg merged 7 commits intomainfrom
retain-refresh-promise-reference

Conversation

@paras-fundwave
Copy link
Collaborator

@paras-fundwave paras-fundwave commented Oct 22, 2024

Description

store reference to currently ongoing refresh-token network calls to avoid making repetitive calls when

  • a lot of calls request token-refreshes simultaneously
  • refreshes take longer than 15s (default wait-time for this lib) - can happen if browser is busy with other calls

Resolutions

Deployment

  • Current pipelines will suffice

@paras-fundwave paras-fundwave added the enhancement New feature or request label Oct 22, 2024
@paras-fundwave paras-fundwave self-assigned this Oct 22, 2024
@paras-fundwave paras-fundwave requested review from rickygarg and removed request for Isha-Sharma October 22, 2024 10:40
@paras-fundwave paras-fundwave marked this pull request as ready for review October 22, 2024 10:40
@rickygarg rickygarg merged commit 62b4e58 into main Oct 22, 2024
@rickygarg rickygarg deleted the retain-refresh-promise-reference branch October 22, 2024 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ENHANCEMENT] repeated calls to refresh-token on parallel bulk execution

3 participants