GitHub user mike-jumper opened a pull request:
https://github.com/apache/guacamole-client/pull/327
GUACAMOLE-524: Correct AuthenticatedUser implementations of Attributes
interface.
These changes correct the JDBC and LDAP implementations of
`AuthenticatedUser` such that `getAttributes()` and `setAttributes()` follow
the definitions of the `Attributes` interface:
* Only attributes which `setAttributes()` will be persisting should be
allowed to be set.
* Name/value pairs given to `setAttributes()`, if accepted, should be
layered on top of existing attributes. They shouldn't replace the entire
attribute map.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mike-jumper/guacamole-client
fix-attributes-contract
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/guacamole-client/pull/327.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #327
----
commit aae17f3a9f0245f08b5bc1f7ac13d4ad2aa49728
Author: Michael Jumper <mjumper@...>
Date: 2018-10-01T19:10:37Z
GUACAMOLE-524: Accept only supported attributes via setAttributes(). Do not
replace existing attributes.
----
---
