Github user krisss85 commented on a diff in the pull request:

    https://github.com/apache/guacamole-client/pull/351#discussion_r247323482
  
    --- Diff: guacamole-docker/bin/start.sh ---
    @@ -404,6 +404,42 @@ END
         ln -s /opt/guacamole/radius/guacamole-auth-*.jar "$GUACAMOLE_EXT"
     }
     
    +## Adds properties to guacamole.properties which select the OPENID
    +## authentication provider, and configure it to connect to the specified 
OPENID
    +## provider.
    +##
    +associate_openid() {
    +
    +    # Verify required parameters are present
    +    if [ -z "$OPENID_AUTHORIZATION_ENDPOINT" ] || \
    +       [ -z "$OPENID_JWKS_ENDPOINT" ]          || \
    +       [ -z "$OPENID_ISSUER" ]                 || \
    +       [ -z "$OPENID_CLIENT_ID" ]              || \          
    +       [ -z "$OPENID_REDIRECT_URI" ]
    +    then
    +        cat <<END
    +FATAL: Missing required environment variables to setup openid 
authentication.
    
+-------------------------------------------------------------------------------
    +END
    --- End diff --
    
    Makes sense of course. Please verify the latest version with amendments in 
this section.


---

Reply via email to