Mailing-List: contact user-help@guacamole.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: user@guacamole.incubator.apache.org
MIME-Version: 1.0
In-Reply-To: <CAEp=6yuFZLOJ5_DANq3Ax=_73VBY4dXF+APcjPgjQPcs2hgVNA@mail.gmail.com>
References: <1504835304397-0.post@n4.nabble.com> <688504838.5334990.1504876087681@mail.yahoo.com>
 <CAEp=6yuFZLOJ5_DANq3Ax=_73VBY4dXF+APcjPgjQPcs2hgVNA@mail.gmail.com>
From: Mike Jumper <mike.jumper@guac-dev.org>
Date: Fri, 8 Sep 2017 09:25:18 -0700
Message-ID: <CALKeL-Mr3csj5ZNOWCJvTERe82A+8m7VmPgGb7MLNp2D=kVgxA@mail.gmail.com>
Subject: Re: Guacamole not reading guacamole.properties
To: user@guacamole.incubator.apache.org
Content-Type: multipart/alternative; boundary="001a114558be3498870558b00802"
archived-at: Fri, 08 Sep 2017 16:26:05 -0000

--001a114558be3498870558b00802
Content-Type: text/plain; charset="UTF-8"

On Fri, Sep 8, 2017 at 8:28 AM, D Chen <perktvacc@gmail.com> wrote:

> Looks like the logs got dropped...Anyways, I did more testing (Alpine 3.6,
> Guacamole 0.9.13) and narrowed it down to the "user-mapping" property
> contained within the guacamole.properties file not being read.
>
> guacamole.properties properties:
>
> guac:/etc/guacamole# cat guacamole.properties
>
> guacd-port: 7777
> user-mapping: /etc/guacamole/map/user-mapping.xml
>
>
>
The manual is incorrect in a couple ways here:

1) The "user-mapping" property doesn't actually exist and will have no
effect. The property is actually "basic-user-mapping":

https://github.com/apache/incubator-guacamole-client/blob/1c0ee41d0ecd5bc4a3550804b74b73b901e074c2/guacamole/src/main/java/org/apache/guacamole/auth/file/FileAuthenticationProvider.java#L72-L84

2) The "basic-user-mapping" property was supposed to be deprecated in favor
of GUACAMOLE_HOME/user-mapping.xml. It shouldn't be documented in the
manual at all:

https://github.com/apache/incubator-guacamole-client/blob/1c0ee41d0ecd5bc4a3550804b74b73b901e074c2/guacamole/src/main/java/org/apache/guacamole/auth/file/FileAuthenticationProvider.java#L128

The manual needs to be updated to remove mention of this property, as (in
addition to being wrong) it shouldn't be used in new deployments. For
reference, the deprecation was due to the development associated with the
initial migration to the Apache Incubator:

https://issues.apache.org/jira/browse/GUACAMOLE-1

And is mentioned in the release notes of the first release under the
Incubator:

http://guacamole.incubator.apache.org/releases/0.9.10-incubating/#deprecation--compatibility-notes

Anyway, as far as your case is concerned, the proper solution would be to
place "user-mapping.xml" within GUACAMOLE_HOME ("/etc/guacamole"). Anything
else leverages deprecated features.

- Mike

--001a114558be3498870558b00802
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div class=3D"gmail_extra"><div class=3D"gmail_quote">On F=
ri, Sep 8, 2017 at 8:28 AM, D Chen <span dir=3D"ltr">&lt;<a href=3D"mailto:=
perktvacc@gmail.com" target=3D"_blank">perktvacc@gmail.com</a>&gt;</span> w=
rote:<br><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8e=
x;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir=3D"ltr"=
><div><div>Looks like the logs got dropped...Anyways, I did more testing (A=
lpine 3.6, Guacamole 0.9.13) and narrowed it down to the &quot;user-mapping=
&quot; property contained within the guacamole.properties file not being re=
ad.</div><div><br></div><div>guacamole.properties properties:</div><div><sp=
an style=3D"color:rgb(0,0,0)"><br></span></div><div><span style=3D"color:rg=
b(0,0,0)">guac:/etc/guacamole# cat guacamole.properties</span><br></div><di=
v><pre style=3D"color:rgb(0,0,0)">guacd-port: 7777
user-mapping: /etc/guacamole/map/user-mappin<wbr>g.xml</pre><pre style=3D"c=
olor:rgb(0,0,0)"><br></pre></div></div></div></blockquote><div><br></div><d=
iv>The manual is incorrect in a couple ways here:</div><div><br></div><div>=
1) The &quot;user-mapping&quot; property doesn&#39;t actually exist and wil=
l have no effect. The property is actually &quot;basic-user-mapping&quot;:<=
/div><div><br></div><div><a href=3D"https://github.com/apache/incubator-gua=
camole-client/blob/1c0ee41d0ecd5bc4a3550804b74b73b901e074c2/guacamole/src/m=
ain/java/org/apache/guacamole/auth/file/FileAuthenticationProvider.java#L72=
-L84">https://github.com/apache/incubator-guacamole-client/blob/1c0ee41d0ec=
d5bc4a3550804b74b73b901e074c2/guacamole/src/main/java/org/apache/guacamole/=
auth/file/FileAuthenticationProvider.java#L72-L84</a><br></div><div><br></d=
iv><div>2) The &quot;basic-user-mapping&quot; property was supposed to be d=
eprecated in favor of GUACAMOLE_HOME/user-mapping.xml. It shouldn&#39;t be =
documented in the manual at all:</div><div><br></div><div><a href=3D"https:=
//github.com/apache/incubator-guacamole-client/blob/1c0ee41d0ecd5bc4a355080=
4b74b73b901e074c2/guacamole/src/main/java/org/apache/guacamole/auth/file/Fi=
leAuthenticationProvider.java#L128">https://github.com/apache/incubator-gua=
camole-client/blob/1c0ee41d0ecd5bc4a3550804b74b73b901e074c2/guacamole/src/m=
ain/java/org/apache/guacamole/auth/file/FileAuthenticationProvider.java#L12=
8</a><br></div><div><br></div><div>The manual needs to be updated to remove=
 mention of this property, as (in addition to being wrong) it shouldn&#39;t=
 be used in new deployments. For reference, the deprecation was due to the =
development associated with the initial migration to the Apache Incubator:<=
/div><div><br></div><div><a href=3D"https://issues.apache.org/jira/browse/G=
UACAMOLE-1">https://issues.apache.org/jira/browse/GUACAMOLE-1</a><br></div>=
<div><br></div><div>And is mentioned in the release notes of the first rele=
ase under the Incubator:</div><div><br></div><div><a href=3D"http://guacamo=
le.incubator.apache.org/releases/0.9.10-incubating/#deprecation--compatibil=
ity-notes">http://guacamole.incubator.apache.org/releases/0.9.10-incubating=
/#deprecation--compatibility-notes</a></div><div><br></div><div>Anyway, as =
far as your case is concerned, the proper solution would be to place &quot;=
user-mapping.xml&quot; within GUACAMOLE_HOME (&quot;/etc/guacamole&quot;). =
Anything else leverages deprecated features.</div><div><br></div><div>- Mik=
e</div><div><br></div></div></div></div>

--001a114558be3498870558b00802--