Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed axios interceptor concurrent calls #1278

Closed
wants to merge 2 commits into from

Conversation

DanutIlie
Copy link
Collaborator

Issue

Dapp is receiving the token before the interceptor is set

Root cause

When the native token is set in useAxiosInterceptorContext, both api call from dapp is triggered and the interceptor request and leads to a concurrency issue

Fix

Remove native token dependency for injecting the interceptor

Contains breaking changes

  • No
  • Yes

Updated CHANGELOG

  • No
  • Yes

Testing

  • User testing
  • Unit tests

@DanutIlie DanutIlie closed this Oct 9, 2024
@arhtudormorar arhtudormorar deleted the di/fix/axios-interceptors-concurrency branch October 9, 2024 12:27
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.

1 participant