Skip to content

Commit

Permalink
fix: stop writing RawClaim into authorization cookie to reduce cookie…
Browse files Browse the repository at this point in the history
… size. Fixes #9530, #10153 (#10170)

Signed-off-by: Kewei Ma <kewei@indeed.com>
Signed-off-by: Alex Collins <alex_collins@intuit.com>
Co-authored-by: Alex Collins <alexec@users.noreply.github.com>
Co-authored-by: Alex Collins <alex_collins@intuit.com>
  • Loading branch information
3 people authored and terrytangyuan committed Mar 29, 2023
1 parent 7e0dc0a commit 5c5c650
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 1 deletion.
1 change: 0 additions & 1 deletion server/auth/sso/sso.go
Expand Up @@ -285,7 +285,6 @@ func (s *sso) HandleCallback(w http.ResponseWriter, r *http.Request) {
Expiry: jwt.NewNumericDate(time.Now().Add(s.expiry)),
},
Groups: groups,
RawClaim: c.RawClaim,
Email: c.Email,
EmailVerified: c.EmailVerified,
ServiceAccountName: c.ServiceAccountName,
Expand Down
1 change: 1 addition & 0 deletions test/e2e/manifests/sso/kustomization.yaml
Expand Up @@ -3,6 +3,7 @@ kind: Kustomization

resources:
- ../../../../manifests/quick-start/sso
- ../mixins/argo-server.service-account-token-secret.yaml

patchesStrategicMerge:
- ../mixins/argo-server-deployment.yaml
Expand Down

0 comments on commit 5c5c650

Please sign in to comment.