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.

Sign up for GitHub

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

Jump to bottom

__cypress.initial cookie expired date causing signin side-effects #29268

Closed
jj497 opened this issue Apr 4, 2024 · 2 comments · May be fixed by #29269
Closed

__cypress.initial cookie expired date causing signin side-effects #29268

jj497 opened this issue Apr 4, 2024 · 2 comments · May be fixed by #29269

Comments

@jj497
Copy link

jj497 commented Apr 4, 2024

Current behavior

Testing Login on a 3rd party website.
__cypress.initial cookie is set with domain of the 3rd party (target) website that expires 1970-01-01T00:00:00.000Z. This cookie is visible under the cookie tab of the login request.
As a result, subsequent calls to complete the login fail because of an expired token.

Since I don't own the login page, I can't update the logic. However, I know this is the issue because
I've retested changing the first visit url to another website in order to change the domain of the __cypress.initial cookie. Login is successful, but the dashboard page takes forever to load.

Desired behavior

set __cypress.initial cookie with an expire property in the future like 2070-01-01T00:00:00.000Z

Test code to reproduce

N/A

Cypress Version

13.7.1

Node version

v21.2.0

Operating System

macOS 12.7.3

Debug Logs

N/A

Other

No response
jj497 added a commit to jj497/cypress that referenced this issue Apr 4, 2024
@jj497
Update response-middleware.ts cookie expiration date
c1c6942 
see  cypress-io#29268
@jj497 jj497 mentioned this issue Apr 4, 2024
Open
3 tasks
@cacieprins
Copy link
Contributor

Hi @jj497 ,

The __cypress.initial cookie is used internally by Cypress, and should not impact your application tokens. You should not have to access this cookie yourself. Can you provide us with a sample reproduction on a fork of cypress-test-tiny or cypress-realworld-app?

Thank you!

@cacieprins cacieprins added stage: needs information Not enough info to reproduce the issue stage: needs investigating Someone from Cypress needs to look at this and removed stage: needs investigating Someone from Cypress needs to look at this labels Apr 4, 2024
@jennifer-shehane
Copy link
Member

We’ll close this issue since a repro has not been provided.

@jennifer-shehane jennifer-shehane closed this as not planned Won't fix, can't repro, duplicate, stale Apr 12, 2024
@jennifer-shehane jennifer-shehane removed the stage: needs information Not enough info to reproduce the issue label Apr 12, 2024
@jj497 jj497 mentioned this issue Apr 20, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

misc: Update response-middleware.ts cookie expiration date jj497/cypress
3 participants
@cacieprins @jennifer-shehane @jj497