Skip to content

modify cookies SameSite property#84

Merged
alcalawil merged 1 commit intodevelopfrom
fix/wil/str-345
Jan 12, 2023
Merged

modify cookies SameSite property#84
alcalawil merged 1 commit intodevelopfrom
fix/wil/str-345

Conversation

@alcalawil
Copy link
Copy Markdown
Contributor

how to test:

run the api,
create user login, delete cookies, login again, make some requests, make sure the auth cookie is still working

@alcalawil alcalawil requested review from auroter and saito-sv January 12, 2023 19:03
Copy link
Copy Markdown
Contributor

@auroter auroter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image
LGTM

@alcalawil alcalawil merged commit 83c56d8 into develop Jan 12, 2023
@alcalawil alcalawil deleted the fix/wil/str-345 branch January 12, 2023 19:10
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.

3 participants