Skip to content

Conversation

@arndom
Copy link
Contributor

@arndom arndom commented Nov 22, 2025

PR Checklist

Please check if your PR fulfills the following requirements:

Bugs / Features

What is the current behavior?

Despite a successful login after being forced to log out, there is an error loop.

What is the new behavior?

On logout, all queries are invalidated, ensuring new data is fetched on login

resolves #6997

Notes for reviewers

@arndom arndom requested a review from a team as a code owner November 22, 2025 14:29
@changeset-bot
Copy link

changeset-bot bot commented Nov 22, 2025

🦋 Changeset detected

Latest commit: 8ddfeda

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@refinedev/core Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@netlify
Copy link

netlify bot commented Nov 22, 2025

Deploy Preview for refine-doc-live-previews ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 8f5da78
🔍 Latest deploy log https://app.netlify.com/projects/refine-doc-live-previews/deploys/6921c8bf154e880008f1888d
😎 Deploy Preview https://deploy-preview-7124--refine-doc-live-previews.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@alicanerdurmaz
Copy link
Member

hey @arndom, this test is failing:

Tests Failed or an Error Occured 
 Failed Examples: 
  |-- auth-keycloak 

Could you take a look?

@arndom
Copy link
Contributor Author

arndom commented Dec 11, 2025

@alicanerdurmaz I locally ran the test suite for the auth-keycloak example, and they all passed.

  auth-keycloak
    login
      √ should login (17329ms)
      √ should redirect to /login if user not authenticated (7790ms)
    logout
      √ should logout (10043ms)
    get identity
      √ should render getIdentity response on header (5869ms)


  4 passing (42s)


  (Results)

  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ Tests:        4                                                                                │
  │ Passing:      4                                                                                │
  │ Failing:      0                                                                                │
  │ Pending:      0                                                                                │
  │ Skipped:      0                                                                                │
  │ Screenshots:  0                                                                                │
  │ Video:        false                                                                            │
  │ Duration:     41 seconds                                                                       │
  │ Spec Ran:     all.cy.ts                                                                        │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘

I noticed the same example fails in the github runner for other unrelated-to-auth PRs:

So it may be an issue within the github runner...could you take a look?

@BatuhanW BatuhanW merged commit dbf5ab8 into refinedev:main Dec 29, 2025
5 checks passed
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.

[BUG] Can't login success after session expired(loop in authProvider onError)

3 participants