petzsch commented on a change in pull request #615:
URL: https://github.com/apache/guacamole-client/pull/615#discussion_r644621457
##########
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
+environment variables:
+
+ SAML_IDP_METADATA_URL The URI of the XML metadata file that from
the SAML Identity
+ Provider
+END
+ exit 1;
+ fi
+
+ # Update config file
+ set_property "saml-idp-metadata-url"
"$SAML_IDP_METADATA_URL"
+ set_optional_property "saml-idp-url" "$SAML_IDP_URL"
+ set_optional_property "saml-entity-id" "$SAML_ENTITY_ID"
+ set_optional_property "saml-callback-url"
"$SAML_CALLBACK_URL"
+ set_optional_property "saml-strict" "$SAML_STRICT"
+ set_optional_property "saml-debug" "$SAML_DEBUG"
+ set_optional_property "saml-compress-request"
"$SAML_COMPRESS_REQUEST"
+ set_optional_property "saml-compress-response"
"$SAML_COMPRESS_RESPONSE"
+ set_optional_property "saml-group-attribute"
"SAML_GROUP_ATTRIBUTE"
Review comment:
should be:
```suggestion
set_optional_property "saml-group-attribute"
"$SAML_GROUP_ATTRIBUTE"
```
##########
File path: guacamole-docker/bin/build-guacamole.sh
##########
@@ -165,6 +165,21 @@ if [ -f extensions/guacamole-auth-duo/target/*.tar.gz ];
then
"*.jar"
fi
+#
+# Copy Saml auth extension if it was built
+#
+
+if [ -f extensions/guacamole-auth-saml/target/*.tar.gz ]; then
+ mkdir -p "$DESTINATION/saml"
+ tar -xzf extensions/guacamole-auth-saml/target/*.tar.gz \
+ -C "$DESTINATION/duo/" \
Review comment:
```suggestion
-C "$DESTINATION/saml/" \
```
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
