Add saml2 profile - #196
Conversation
677a5c9 to
fd0a155
Compare
| @@ -0,0 +1,3 @@ | |||
| PULP_SAML_CONFIG='{saml_config}' | |||
| PULP_LOGIN_REDIRECT_URL='/pulp/default/api/v3/login/' | |||
There was a problem hiding this comment.
Does this only work with domains on?
There was a problem hiding this comment.
I guess so.
But after all if this points to nowhere it's more like an inconvenience. These settings serve as an example (and not even a particularly good one) for setting up SAML support.
Is there a way to make the profile more flexible?
| - "/tmp/.X11-unix/:/tmp/.X11-unix/:ro" | ||
| environment: | ||
| DISPLAY: ":0.0" | ||
| default: false |
There was a problem hiding this comment.
is default suppose to be under networks? I get this error trying to compose up this:
saml2_compose.yaml: services.browser additional properties 'default' not allowed
There was a problem hiding this comment.
No i think it was supposed to default off the whole container. So you needed to start it purposefully. But that doesn't even do it, right?
There was a problem hiding this comment.
If you want a service that doesn't run automatically and needs to be specified to run, then use profiles: https://docs.docker.com/compose/how-tos/profiles/
There was a problem hiding this comment.
Profiles don't seem to work with oci-env now. I'm just leaving it for today.
|
@mdellweg Rebase to get green CI then I think we will be good to go. |
2 participants
No description provided.