[GitHub] [guacamole-client] jbpaux commented on a change in pull request #615: GUACAMOLE-1322: Include SAML Plugin in Docker Image

[GitBox <gi...@apache.org>]

jbpaux commented on a change in pull request #615:
URL: https://github.com/apache/guacamole-client/pull/615#discussion_r742256600



##########
File path: guacamole-docker/bin/start.sh
##########
@@ -594,6 +594,46 @@ END
 
 }
 
+## Adds properties to guacamole.properties which select the SAML
+## authentication provider, and configure it to connect to the specified SAML
+## provider.
+##
+associate_saml() {
+
+    # Verify required parameters are present
+    if [ -z "$SAML_IDP_METADATA_URL" ]
+    then
+        cat <<END
+FATAL: Missing required environment variables
+-------------------------------------------------------------------------------
+If using an openid authentication, you must provide each of the following

Review comment:
       fixed :)




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@guacamole.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org