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

[Bug]: setting cookies for some GET requests causes session problems #136

Closed
1 task done
samberrry opened this issue Aug 12, 2023 · 1 comment
Closed
1 task done

Comments

@samberrry
Copy link
Member

samberrry commented Aug 12, 2023

Actual Behavior

When you log into the system, and then refresh home page or any other page, the user will be logged out. But this scenario happens when PrestaShop sets two cookies instead of a cookie for sessions. So, the wishlist, cart, addresses, and other GET requests will replace wrong cookie and the customer will log out..

setting cookies for cart and wishlist get requests causes session problems

Possible Solution

No response

Steps to reproduce

No response

What version of PrestaShop integration are you using?

any

What version of Node.js are you using?

any

What browser (and version) are you using?

any

What operating system (and version) are you using?

any

Relevant log output

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@samberrry samberrry added bug Something isn't working v1.3.0 labels Aug 12, 2023
@samberrry samberrry changed the title [Bug]: setting cookies for cart and wishlist get requests causes session problems [Bug]: setting cookies for cart and wishlist, addresses GET requests causes session problems Aug 12, 2023
@samberrry samberrry changed the title [Bug]: setting cookies for cart and wishlist, addresses GET requests causes session problems [Bug]: setting cookies for some GET requests causes session problems Aug 12, 2023
@samberrry
Copy link
Member Author

This issue written when I got the error:
#137
but fix related to the current issue will be a refinement not a solution to the issue 137.

@samberrry samberrry added refactor and removed bug Something isn't working labels Aug 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant