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: allow for overwritting the cookie-domain flag on the oidc proxy #157

Merged
merged 21 commits into from
Oct 25, 2024

Conversation

jakeyheath
Copy link
Contributor

@jakeyheath jakeyheath commented Oct 24, 2024

https://czi.atlassian.net/browse/VC-1158

Summary

Some folks require that the cookie be set on a different domain other than the default domain argus provides and the domains that are used to host the application. For example, some applications need to be able to set the cookie on a parent domain so it can be easily shared with other subdomains.

The following configuration will overwrite the default --cookie-domain flag with myproddomain.com:

stack:
  global:
    oidcProxy:
      enabled: true
      extraArgs:
        - --cookie-domain=myproddomain.com

Subdomains of myproddomain.com will have access to this cookie.

@jakeyheath jakeyheath marked this pull request as ready for review October 24, 2024 22:22
Base automatically changed from CCIE-3528-conditional-logic-around-the-ingress-yml-to-point-to-the-service-or-the-oauth-2-proxy-service to CCIE-3526-add-upstream-flags-to-the-oauth-2-proxy-deployment October 25, 2024 16:43
Base automatically changed from CCIE-3526-add-upstream-flags-to-the-oauth-2-proxy-deployment to CCIE-3527-add-the-skip-auth-values-to-the-skip-auth-flags-of-the-oauth-2-proxy-deployment October 25, 2024 16:43
Base automatically changed from CCIE-3527-add-the-skip-auth-values-to-the-skip-auth-flags-of-the-oauth-2-proxy-deployment to main October 25, 2024 17:02
@jakeyheath jakeyheath merged commit 23a7471 into main Oct 25, 2024
14 checks passed
@jakeyheath jakeyheath deleted the CCIE-3548-enable-login-and-logout-behavior-for-vcp branch October 25, 2024 17:09
@czi-github-helper czi-github-helper bot mentioned this pull request Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants