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

feat: support setting uc openapi session #6347

Merged
merged 1 commit into from
May 15, 2024

Conversation

chengjoey
Copy link
Contributor

What this PR does / why we need it:

support setting uc openapi session

Specified Reviewers:

/assign @sfwn

ChangeLog

Language Changelog
🇺🇸 English support setting uc openapi session
🇨🇳 中文 支持uc openapisession的配置

Need cherry-pick to release versions?

Add comment like /cherry-pick release/1.0 when this PR is merged.

For details on the cherry pick process, see the cherry pick requests section under CONTRIBUTING.md.

@erda-bot erda-bot requested a review from sfwn May 15, 2024 03:51
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."`
Copy link
Member

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."`
Copy link
Member

Choose a reason for hiding this comment

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

same

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@sfwn
Copy link
Member

sfwn commented May 15, 2024

/approve

@sfwn sfwn merged commit d3625c4 into erda-project:master May 15, 2024
28 checks passed
chengjoey added a commit to chengjoey/erda that referenced this pull request May 15, 2024
support setting uc openapi session
erda-bot pushed a commit that referenced this pull request May 15, 2024
support setting uc openapi session
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

3 participants