-
Notifications
You must be signed in to change notification settings - Fork 44
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
Environment variables #180
Comments
Good point. I didn’t realize those never got documented properly. The |
What's the different use for SIGN_SECRET and ENCRYPT_SECRET? Do we need both for everything? eg: oauth2-proxy only uses a single |
Here's the breakdown:
|
Thanks for keeping security in mind! |
In one of my m2m use case, I use jwt plugin and all requests will come with jwt header:
In this case, I guess there are no cookie, sign/encrypt secrets needed, so I can just set those env variables to dummy values? |
Yeah, but you should not need a new deployment per app if that concept was not clear. It's designed to have a single installation to handle many many configs/apps. |
That makes sense.
Is this key cached? If so, for how long? Is this configurable. I see there is "expiresIn" option in this doc but not sure if that is related to this key caching. The provider may update/rotate this key, though infrequent, which will cause jwt validation failure. Will you re-retrieve the key and try again in that case, or will you always re-retrieve this file and update in-memory cache at some pre-defined internal? |
Sorry, been super busy. So the key is cached yes and it's based on the header sent by the provider. In short it will 'just work'. |
I checked the key from iap :
Does this mean you will cache it for |
https://github.com/auth0/node-jwks-rsa/blob/master/EXAMPLES.md#caching I want to say that doc has changed from what it was the last time I looked at it so it may be slightly different. The documented behavior there (which may not be the same version currently imported into the app) seems to indicated a static value defaulting to 10m. I was under the impression the older versions were using one of those 2 headers but don't recall which. |
My understanding is each request to the nginx ingress will trigger a call to external-auth-server. I plan to have one external-auth-server installation for each Kubernetes cluster (nginx ingress) where clusters are scattered across the world, so that auth traffic are local within the cluster. Having all clusters passing every request to a central external-auth-server installation may not be optimal. That's why I am hoping to see oauth plugin supports redirect_uri with uri only without base dns name. The base dns name could be retrieved to $host header if not provided in the redirect_uri config. |
Yeah fair enough. The context of that comment was more about multiple apps within a single cluster. I can look into making the value of that property a handlebars template. |
The docker example in the README mentions bunch of environment variables.
Is there a doc explaining those? I assume some of them are used only by some plugins. So if I don't use those plugins, I could just not set, or give some dummy values.
The text was updated successfully, but these errors were encountered: