Skip to content

Conversation

@bsnuggs1
Copy link

@bsnuggs1 bsnuggs1 commented Sep 17, 2020

…vironment variables.

This change allows majority of the SAML environment variables to be configured via the environment parameter in docker.

I created this change to offer an alternative solution to #242

@timabbott
Copy link
Member

@bsnuggs1 can you fix the shellcheck errors flagged by CI?

@bsnuggs1 bsnuggs1 force-pushed the feature/entrypoint-settings-object-type-support branch from 0404e09 to 1071c6b Compare September 18, 2020 04:25
@bsnuggs1
Copy link
Author

Sure thing, I applied the fix just now.

@timabbott
Copy link
Member

Can you show the output of what this generates in a sample case? I'm not convinced that the escaping will actually work correctly.

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.

2 participants