-
Notifications
You must be signed in to change notification settings - Fork 375
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
feat: support setting uc openapi session #6347
Conversation
UCRedirectAddrs []string `file:"uc_redirect_addrs"` | ||
SessionCookieName string `file:"session_cookie_name"` | ||
SessionCookieDomains []string `file:"session_cookie_domain"` | ||
CookieMaxAge time.Duration `file:"cookie_max_age" default:"24h" desc:"max age of the CSRF cookie. optional."` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not csrf token
SessionCookieDomains []string `file:"session_cookie_domain"` | ||
CookieMaxAge time.Duration `file:"cookie_max_age" default:"24h" desc:"max age of the CSRF cookie. optional."` | ||
// CookieSameSite default set to 2, which is `lax`, more options see https://github.com/golang/go/blob/619b419a4b1506bde1aa7e833898f2f67fd0e83e/src/net/http/cookie.go#L52-L57 | ||
CookieSameSite int `file:"cookie_same_site" default:"2" desc:"indicates if CSRF cookie is SameSite. optional."` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
a0c432d
to
dfc8f7b
Compare
/approve |
support setting uc openapi session
What this PR does / why we need it:
support setting uc openapi session
Specified Reviewers:
/assign @sfwn
ChangeLog
Need cherry-pick to release versions?
Add comment like
/cherry-pick release/1.0
when this PR is merged.