fix:(middlewares.go) - Set Cookie SameSite mode to Strict - 1776 (#1777)

* None is deprecated and will fallback to Lax in the future.
* Using Strict is future proof and provides additional CSR protection

Signed-off-by: Manuel Kroeber <manuel.kroeber@gmail.com>

Signed-off-by: Manuel Kroeber <manuel.kroeber@gmail.com>
This commit is contained in:
Manuel 2022-09-27 23:58:47 +02:00 committed by Joe Stump
parent 822c364645
commit e81048135b
No known key found for this signature in database
GPG Key ID: 29151C3EC48A0EB9
2 changed files with 2 additions and 1 deletions

View File

@ -112,7 +112,7 @@ func clientUniqueIdAdder(next http.Handler) http.Handler {
MaxAge: consts.CookieExpiry,
HttpOnly: true,
Secure: true,
SameSite: http.SameSiteNoneMode,
SameSite: http.SameSiteStrictMode,
Path: "/",
}
http.SetCookie(w, c)

View File

@ -161,6 +161,7 @@ func getPlayer(players core.Players) func(next http.Handler) http.Handler {
Value: player.ID,
MaxAge: consts.CookieExpiry,
HttpOnly: true,
SameSite: http.SameSiteStrictMode,
Path: "/",
}
http.SetCookie(w, cookie)