Skip to content

Fix auto logout issues #881

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

Open
pftom opened this issue May 12, 2025 · 1 comment
Open

Fix auto logout issues #881

pftom opened this issue May 12, 2025 · 1 comment
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@pftom
Copy link
Contributor

pftom commented May 12, 2025

Describe the bug
A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

  • OS: [e.g. iOS]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

Smartphone (please complete the following information):

  • Device: [e.g. iPhone6]
  • OS: [e.g. iOS8.1]
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 22]

Additional context
Add any other context about the problem here.

@pftom pftom added bug Something isn't working good first issue Good for newcomers labels May 12, 2025
@luojiyin1987
Copy link
Contributor

One-time Refresh Token mechanism conflict ?
https:github.com/refly-ai/refly/blob/665afe959d5ed37b43fa914a497cdf64f36f2658/apps/api/src/modules/auth/auth.service.ts#L130-L134

Token race condition caused by concurrent API requests

if (response?.status === 401) {
try {
const retryResponse = await refreshTokenAndRetry(request);
return retryResponse;
} catch (error) {
if (error instanceof AuthenticationExpiredError) {
await logout();
}
return response;
}
}

Conflict between scheduled tasks and token expiration time

refetchInterval: 15 * 1000, // Refetch every 15 seconds

Implement multi-tab Token synchronization, enhance error handling and retry mechanisms.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants