GitHub user mike-jumper opened a pull request:
https://github.com/apache/incubator-guacamole-client/pull/61
GUACAMOLE-77: Ensure non-existent user accounts are not displayed.
This change establishes the distinction within the REST API between the
user object *representing* the current user (`.../self`) and the user object
representing an *existing* user (`.../users/{username}`).
This distinction has existed within `UserContext` for some time, but was
never exposed within the REST API. It's an important distinction, as just
because a backend agrees on a user's identity does not mean the user actually
exists within the backend.
By implementing the above change (44d470e), and by not listing
pseudo-accounts within the user directory of the sharing auth provider
(7039dad), the non-relevant pseudo-accounts are no longer displayed as options
in the admin interface. In case they ever *do* get displayed somewhere
unexpectedly, they will at least be readable, as I've also added actual
translation strings (87b517f).
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mike-jumper/incubator-guacamole-client
account-visibility
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-guacamole-client/pull/61.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 #61
----
commit 87b517ff3e994cd1394e041bb2d2b14e4b32686e
Author: Michael Jumper <[email protected]>
Date: 2016-08-20T20:12:04Z
GUACAMOLE-77: Add translation strings for MySQL/PostgreSQL shared
connection data source.
commit 44d470e6ac8f47e587f8822100dbd54d14c9603a
Author: Michael Jumper <[email protected]>
Date: 2016-08-20T20:54:20Z
GUACAMOLE-77: Distinguish within REST between the user's
identity/permissions and actual existing user objects.
commit 7039dad32a82187ad30389bac4f9cb0418c471f5
Author: Michael Jumper <[email protected]>
Date: 2016-08-20T20:55:00Z
GUACAMOLE-77: Users do not exist within the database auth's connection
sharing system.
commit 78cda2ac171e2c5ca12c4f721b5d9612c62dcaf8
Author: Michael Jumper <[email protected]>
Date: 2016-08-20T21:23:43Z
GUACAMOLE-77: Retrieve the current user's permissions via ".../self" rather
than ".../users/{username}". The current user may not actually exist.
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
