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

    https://github.com/apache/guacamole-server/pull/205#discussion_r236386735
  
    --- Diff: src/guacd/conf-file.c ---
    @@ -188,15 +188,21 @@ guacd_config* guacd_conf_load() {
         conf->key_file = NULL;
     #endif
     
    +    /* Determine path of configuration file */
    +    if(conf_file_path == NULL) {
    +        conf_file_path = getenv("GUACD_CONF_FILE");
    --- End diff --
    
    Why the change from the constant `GUACD_CONF_FILE` to 
`getenv("GUACD_CONF_FILE")`?  I believe this changes from using the constant to 
assuming that it is defined within the environment??


---

Reply via email to