Github user jaredfrees commented on a diff in the pull request: https://github.com/apache/guacamole-client/pull/299#discussion_r194876880 --- Diff: guacamole-ext/src/main/java/org/apache/guacamole/net/auth/AbstractAuthenticatedUser.java --- @@ -41,4 +42,12 @@ public void invalidate() { // Nothing to invalidate } + public Map<String, String> getAttributes() { --- End diff -- I think Option 2 is the way to go. Personally, I don't think we should mess so much with the other extensions.
---