-
Notifications
You must be signed in to change notification settings - Fork 147
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
Set strong securityContext by default #145
Conversation
Signed-off-by: Alexandre Desjardins <alexandre.bd@tutanota.com>
Signed-off-by: Alexandre Desjardins <alexandre.bd@tutanota.com>
Signed-off-by: Alexandre Desjardins <alexandre.bd@tutanota.com>
@jemag This is a breaking change that cannot be introduced this way. |
@pierluigilenoci Do you have any suggestions on how you would like this change to be introduced? This should not be breaking previous installations and has not in my testing, but let me know if you think I missed something. |
@jemag In reality, I don't feel very confident either to propose a possible solution. @desaintmartin @tlawrie @NickMeves @JoelSpeed Would you be so kind as to give us an idea on how to deal with this? |
I applied these values and encountered no breaking changes. Thanks - seems like a great idea. |
Fixes #142
I have been using this
securityContext
with 3 different oauth-proxy so far and have not encountered any issues. Would still appreciate someone other than me testing it as well.