rp
09/04/2021, 2:12 PMapiBasePath
config) to the path.
You can do the following:
- Override the signout API on the backend to have the logic of your /logout API. OR;
- Use the preAPIHook
config to change the path where the request is going.ZeferiniX
09/04/2021, 2:22 PMSIGN_OUT
correct?rp
09/04/2021, 2:32 PMrp
09/04/2021, 2:32 PMrp
09/04/2021, 2:32 PMZeferiniX
09/04/2021, 2:42 PMsignout
in the backend has been disabled so we think that's contributing to the issuerp
09/04/2021, 2:42 PMZeferiniX
09/04/2021, 2:42 PM/signout
endpoint, it gets a 404, thus the cookie remained 🤷‍♂️rp
09/04/2021, 2:42 PMrp
09/04/2021, 2:43 PMrp
09/04/2021, 2:43 PMZeferiniX
09/04/2021, 2:43 PMrp
09/04/2021, 2:43 PMZeferiniX
09/04/2021, 2:44 PMZeferiniX
09/04/2021, 2:45 PMrp
09/05/2021, 6:43 AMZeferiniX
09/05/2021, 11:41 AMsignOut()
then login again, the behavior of the sIRTFrontend
cookie doesn't seem right.
to reproduce:
1. Login
2. Signout
3. Login again
from what we've observed, when 1
happens, the sIRTFrontend
cookie gets set. Upon doing 2
the cookie gets set to remove
. However when doing 3
, the sIRTFrontend
value remains as remove
then when performing isAuthenticated()
, it always returns false
because `sIRTFrontend`'s value is remove
even though we have the access and refresh token cookies from the successful login.
Seems like the culprit is here:
https://github.com/supertokens/supertokens-website/blob/85e9f69bb7a17b6f807e5a1e457fc9163463a6e3/lib/ts/fetch.ts#L599
when the session has been created, I think sIRTFrontend
should be set again. That, or remove that cookie as well during signOut()
🤷‍♂️
we're currently getting away with the issue by doing the latter, which is to remove the sIRTFrontend
cookie manually as well after signOut()
.
are we doing something wrong somehow?rp
09/05/2021, 11:51 AMZeferiniX
09/05/2021, 12:48 PMrp
09/05/2021, 12:48 PMrp
09/05/2021, 12:49 PMZeferiniX
09/05/2021, 12:50 PMrp
09/05/2021, 12:50 PMZeferiniX
09/05/2021, 12:50 PMrp
09/05/2021, 12:50 PMZeferiniX
09/05/2021, 12:50 PMZeferiniX
09/05/2021, 12:50 PMrp
09/05/2021, 12:50 PMrp
09/05/2021, 12:51 PMrp
09/05/2021, 12:51 PM