Skip to content

fix(oauth): remove security threats, and update cookie config#15

Merged
shree-ni merged 4 commits intodevfrom
fix/phase-3
Oct 28, 2025
Merged

fix(oauth): remove security threats, and update cookie config#15
shree-ni merged 4 commits intodevfrom
fix/phase-3

Conversation

@bprahul017
Copy link
Contributor

@bprahul017 bprahul017 commented Oct 28, 2025

####Changes

  • Removed setRefreshCookie, appendTokensOnRedirect configs
  • Set httpOnly false for cookie as default
  • Update seting cookie on refresh:false case

Reviewer

@bprahul017 bprahul017 requested a review from shree-ni October 28, 2025 06:36
shree-ni
shree-ni approved these changes Oct 28, 2025
View reviewed changes
Copy link
Contributor

@shree-ni shree-ni left a comment

Choose a reason for hiding this comment

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

remove emogies

@shree-ni shree-ni merged commit 4d0fe27 into dev Oct 28, 2025
1 check passed
@bprahul017 bprahul017 deleted the fix/phase-3 branch October 28, 2025 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shree-ni shree-ni shree-ni approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bprahul017 @shree-ni