oauth2-proxy / manifests

For hosting manifests to allow for the deployment of OAuth2-Proxy/OAuth2-Proxy
Apache License 2.0
170 stars 153 forks source link

extraArgs need to be itemize #101

Closed zethis closed 1 year ago

zethis commented 2 years ago

extraArgs seems to be mandatory to be used with google provider.

It would be cool to have all the extraArgs details in the documentation:

Here is my configuration to have it working with k8s:

        - --cookie-domain=.domain2.com
        - --cookie-domain=.domain1.com
        - --cookie-secure=false
        - --email-domain=*
        - --provider=google
        - --whitelist-domain=.domain2.com
        - --whitelist-domain=.domain1.com

Plus: I'm not able to have multi whitelist-domain entry with helm. I'm not able to have multi hosts in ingress.

pierluigilenoci commented 2 years ago

Feel free to open a PR.

kladiv commented 2 years ago

+1 oauth2-proxy helm chart seems not accept multiple cookie-domain and multiple whitelist-domain

zethis commented 2 years ago

+1 oauth2-proxy helm chart seems not accept multiple cookie-domain and multiple whitelist-domain

You need to use config.configFile to do that.

Here an example:

config:
  configFile: |-
    email_domains = [ "*" ]
    upstreams = [ "file:///dev/null" ]
    cookie_secure = "false"
    cookie_domains = [ ".domain.com", ".otherdomain.io" ]
    whitelist_domains = [ ".domain.com", ".otherdomain.io"]
    provider = "google"
pierluigilenoci commented 2 years ago

@zethis would you like to do a PR to integrate documentation?

pierluigilenoci commented 2 years ago

@zethis any update on this?

pierluigilenoci commented 1 year ago

Closed with #114