[GitHub] incubator-guacamole-client pull request #191: GUACAMOLE-394: Refactor extens...

2017-09-27 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/191#discussion_r141450769 --- Diff: guacamole-ext/src/main/java/org/apache/guacamole/net/auth/ActivityRecordSet.java --- @@ -0,0 +1,128

[GitHub] incubator-guacamole-client pull request #191: GUACAMOLE-394: Refactor extens...

2017-09-25 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/191#discussion_r140896419 --- Diff: guacamole-ext/src/main/java/org/apache/guacamole/net/auth/ActivityRecordSet.java --- @@ -0,0 +1,128

[GitHub] incubator-guacamole-client pull request #176: GUACAMOLE-346: Avoid blocking ...

2017-07-19 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/176#discussion_r128304961 --- Diff: doc/guacamole-playback-example/src/main/webapp/index.html --- @@ -28,10 +28,19

[GitHub] incubator-guacamole-client pull request #173: GUACAMOLE-338: Expand connecti...

2017-07-12 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/173#discussion_r127118531 --- Diff: guacamole/src/main/webapp/app/groupList/types/GroupListItem.js --- @@ -187,7 +187,7 @@ angular.module('groupList').factory

[GitHub] incubator-guacamole-server pull request #95: GUACAMOLE-39: Do not abort VNC/...

2017-06-13 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-server/pull/95#discussion_r121854232 --- Diff: src/common-ssh/ssh.c --- @@ -285,6 +285,38 @@ static void guac_common_ssh_kbd_callback(const char *name, int name_len

[GitHub] incubator-guacamole-server pull request #95: GUACAMOLE-39: Do not abort VNC/...

2017-06-13 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-server/pull/95#discussion_r121854188 --- Diff: src/common-ssh/common-ssh/ssh.h --- @@ -93,12 +94,17 @@ void guac_common_ssh_uninit(); * @param user

[GitHub] incubator-guacamole-client pull request #158: GUACAMOLE-292: Restrict editin...

2017-05-26 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/158#discussion_r118812447 --- Diff: extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc/base

[GitHub] incubator-guacamole-client pull request #158: GUACAMOLE-292: Restrict editin...

2017-05-26 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/158#discussion_r118812426 --- Diff: guacamole/src/main/webapp/app/manage/controllers/manageUserController.js --- @@ -261,6 +261,31 @@ angular.module('manage

[GitHub] incubator-guacamole-client pull request #158: GUACAMOLE-292: Restrict editin...

2017-05-26 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/158#discussion_r118810040 --- Diff: guacamole/src/main/webapp/app/form/directives/form.js --- @@ -163,6 +181,59 @@ angular.module('form').directive('guacForm

[GitHub] incubator-guacamole-client pull request #158: GUACAMOLE-292: Restrict editin...

2017-05-26 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/158#discussion_r118810011 --- Diff: guacamole/src/main/webapp/app/manage/controllers/manageUserController.js --- @@ -225,8 +225,8 @@ angular.module('manage

[GitHub] incubator-guacamole-client pull request #158: GUACAMOLE-292: Restrict editin...

2017-05-26 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/158#discussion_r118809952 --- Diff: guacamole/src/main/webapp/app/manage/controllers/manageUserController.js --- @@ -261,6 +261,31 @@ angular.module('manage

[GitHub] incubator-guacamole-client pull request #158: GUACAMOLE-292: Restrict editin...

2017-05-26 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/158#discussion_r118809943 --- Diff: extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc/base

[GitHub] incubator-guacamole-server pull request #92: GUACAMOLE-280: Add support for ...

2017-05-26 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-server/pull/92#discussion_r118809754 --- Diff: src/terminal/display.c --- @@ -118,7 +118,7 @@ int __guac_terminal_set_colors(guac_terminal_display* display

[GitHub] incubator-guacamole-client pull request #152: GUACAMOLE-281: Consider templa...

2017-05-07 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/152#discussion_r115174709 --- Diff: guacamole-docker/bin/start.sh --- @@ -427,15 +427,15 @@ fi # Validate that at least one authentication backend

[GitHub] incubator-guacamole-client pull request #152: GUACAMOLE-281: Consider templa...

2017-05-07 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/152#discussion_r115174505 --- Diff: guacamole-docker/bin/start.sh --- @@ -427,15 +427,15 @@ fi # Validate that at least one authentication backend

[GitHub] incubator-guacamole-server pull request #88: GUACAMOLE-278: Add support for ...

2017-04-26 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-server/pull/88#discussion_r113616225 --- Diff: src/terminal/terminal_handlers.c --- @@ -967,6 +967,18 @@ int guac_terminal_csi(guac_terminal* term, unsigned char c

[GitHub] incubator-guacamole-server pull request #89: GUACAMOLE-265: Set connection n...

2017-04-26 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-server/pull/89#discussion_r113614127 --- Diff: src/terminal/terminal_handlers.c --- @@ -1196,6 +1226,10 @@ int guac_terminal_osc(guac_terminal* term, unsigned char c

[GitHub] incubator-guacamole-server pull request #86: GUACAMOLE-278: Add support for ...

2017-04-26 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-server/pull/86#discussion_r113610589 --- Diff: src/terminal/terminal_handlers.c --- @@ -779,13 +919,30 @@ int guac_terminal_csi(guac_terminal* term, unsigned char c

[GitHub] incubator-guacamole-server pull request #86: GUACAMOLE-278: Add support for ...

2017-04-26 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-server/pull/86#discussion_r113608643 --- Diff: src/terminal/display.c --- @@ -127,24 +104,27 @@ int __guac_terminal_hash_codepoint(int codepoint) { int

[GitHub] incubator-guacamole-client pull request #143: GUACAMOLE-250: Export client s...

2017-04-14 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/143#discussion_r111638369 --- Diff: guacamole-common-js/src/main/webapp/modules/Client.js --- @@ -151,39 +153,53 @@ Guacamole.Client = function(tunnel

[GitHub] incubator-guacamole-client pull request #139: GUACAMOLE-250: Add support for...

2017-04-11 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/139#discussion_r111062649 --- Diff: guacamole-common-js/src/main/webapp/modules/Client.js --- @@ -133,6 +133,124 @@ Guacamole.Client = function(tunnel

[GitHub] incubator-guacamole-client pull request #131: GUACAMOLE-244: Support configu...

2017-03-20 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/131#discussion_r107064307 --- Diff: extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/DereferenceAliasesProperty.java --- @@ -0,0

[GitHub] incubator-guacamole-client pull request #131: GUACAMOLE-244: Support configu...

2017-03-20 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/131#discussion_r107064241 --- Diff: extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/ConfigurationService.java --- @@ -216,11

[GitHub] incubator-guacamole-client pull request #131: GUACAMOLE-244: Support configu...

2017-03-20 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/131#discussion_r107064203 --- Diff: extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/DereferenceAliasesMode.java --- @@ -0,0 +1,74

[GitHub] incubator-guacamole-server pull request #76: GUACAMOLE-231: Report server-si...

2017-03-13 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-server/pull/76#discussion_r105828074 --- Diff: src/common/cursor.c --- @@ -295,25 +209,19 @@ void guac_common_cursor_set_argb(guac_common_cursor* cursor, int hx, int hy

[GitHub] incubator-guacamole-server pull request #76: GUACAMOLE-231: Report server-si...

2017-03-13 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-server/pull/76#discussion_r105828607 --- Diff: src/common/cursor.c --- @@ -295,25 +209,19 @@ void guac_common_cursor_set_argb(guac_common_cursor* cursor, int hx, int hy

[GitHub] incubator-guacamole-client pull request #125: GUACAMOLE-225: Bump version nu...

2017-02-28 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/125#discussion_r103522286 --- Diff: extensions/guacamole-auth-duo/pom.xml --- @@ -209,19 +209,11

[GitHub] incubator-guacamole-client pull request #125: GUACAMOLE-225: Bump version nu...

2017-02-28 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/125#discussion_r103519319 --- Diff: extensions/guacamole-auth-duo/pom.xml --- @@ -209,19 +209,11

[GitHub] incubator-guacamole-server pull request #59: GUACAMOLE-200: Do not allow rea...

2017-02-15 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-server/pull/59#discussion_r101442133 --- Diff: src/protocols/rdp/guac_rdpdr/rdpdr_printer.c --- @@ -373,21 +110,25 @@ void guac_rdpdr_process_print_job_write

[GitHub] incubator-guacamole-server pull request #42: GUACAMOLE-176: Support layer lo...

2017-01-24 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-server/pull/42#discussion_r97692703 --- Diff: src/common/common/surface.h --- @@ -317,10 +363,77 @@ void guac_common_surface_clip(guac_common_surface* surface, int x, int

[GitHub] incubator-guacamole-server pull request #39: GUACAMOLE-171: Move broadcast s...

2017-01-24 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-server/pull/39#discussion_r97635653 --- Diff: src/libguac/client.c --- @@ -544,12 +273,12 @@ int guac_client_add_user(guac_client* client, guac_user* user, int argc, char

[GitHub] incubator-guacamole-client pull request #102: GUACAMOLE-36: Add support for ...

2017-01-05 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/102#discussion_r94854369 --- Diff: extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc/user

[GitHub] incubator-guacamole-client pull request #101: GUACAMOLE-36: Add support for ...

2017-01-05 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/101#discussion_r94816705 --- Diff: extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc/security

[GitHub] incubator-guacamole-client pull request #99: GUACAMOLE-36: Support fully tra...

2017-01-03 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/99#discussion_r94526966 --- Diff: guacamole/src/main/webapp/app/rest/types/Error.js --- @@ -43,6 +43,15 @@ angular.module('rest').factory('Error', [function

[GitHub] incubator-guacamole-client pull request #99: GUACAMOLE-36: Support fully tra...

2017-01-03 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/99#discussion_r94526301 --- Diff: guacamole/src/main/webapp/app/rest/types/Error.js --- @@ -43,6 +43,15 @@ angular.module('rest').factory('Error', [function

[GitHub] incubator-guacamole-client pull request #99: GUACAMOLE-36: Support fully tra...

2017-01-03 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/99#discussion_r94525332 --- Diff: guacamole-ext/src/main/java/org/apache/guacamole/language/Translatable.java --- @@ -0,0 +1,41 @@ +/* + * Licensed

[GitHub] incubator-guacamole-client pull request #94: GUACAMOLE-136: Add support for ...

2016-12-05 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/94#discussion_r91015645 --- Diff: extensions/guacamole-auth-duo/src/main/java/org/apache/guacamole/auth/duo/DuoAuthenticationProvider.java --- @@ -0,0 +1,100

[GitHub] incubator-guacamole-client pull request #94: GUACAMOLE-136: Add support for ...

2016-12-05 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/94#discussion_r91014257 --- Diff: extensions/guacamole-auth-duo/src/main/java/org/apache/guacamole/auth/duo/AuthenticationProviderService.java --- @@ -0,0

[GitHub] incubator-guacamole-client pull request #89: GUACAMOLE-133: Document all dep...

2016-12-02 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/89#discussion_r90749865 --- Diff: guacamole/src/licenses/dependencies/jsr250-api-1.0/CDDLv1.0.txt --- @@ -0,0 +1,119 @@ +COMMON DEVELOPMENT

[GitHub] incubator-guacamole-client pull request #78: GUACAMOLE-70: Allow access to b...

2016-11-11 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/78#discussion_r87678304 --- Diff: extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc

[GitHub] incubator-guacamole-server pull request #24: GUACAMOLE-107 Remove references...

2016-10-26 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-server/pull/24#discussion_r85266312 --- Diff: README --- @@ -4,15 +4,11 @@ This README

[GitHub] incubator-guacamole-manual pull request #17: GUACAMOLE-88: Update client plu...

2016-10-19 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-manual/pull/17#discussion_r84208938 --- Diff: src/chapters/adding-protocol.xml --- @@ -118,385 +221,564 @@ AM_CFLAGS = -Werror -Wall -pedantic lib_LTLIBRARIES

[GitHub] incubator-guacamole-manual pull request #17: GUACAMOLE-88: Update client plu...

2016-10-19 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-manual/pull/17#discussion_r84207528 --- Diff: src/chapters/adding-protocol.xml --- @@ -118,385 +221,564 @@ AM_CFLAGS = -Werror -Wall -pedantic lib_LTLIBRARIES

[GitHub] incubator-guacamole-client pull request #71: GUACAMOLE-104: Sanitize filenam...

2016-09-07 Thread jmuehlner
GitHub user jmuehlner opened a pull request: https://github.com/apache/incubator-guacamole-client/pull/71 GUACAMOLE-104: Sanitize filenames before upload or download. To make tomcat happy, we can just replace slashes with something else - in this case underscores. You can merge

[GitHub] incubator-guacamole-client pull request #68: add additional search filter fu...

2016-09-03 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/68#discussion_r77439985 --- Diff: extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/AuthenticationProviderService.java

[GitHub] incubator-guacamole-client pull request #68: add additional search filter fu...

2016-09-03 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/68#discussion_r77439971 --- Diff: extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/AuthenticationProviderService.java

[GitHub] incubator-guacamole-client pull request #68: add additional search filter fu...

2016-09-03 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/68#discussion_r77439888 --- Diff: extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/AuthenticationProviderService.java

[GitHub] incubator-guacamole-client pull request #68: add additional search filter fu...

2016-09-03 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/68#discussion_r77439884 --- Diff: extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/AuthenticationProviderService.java

[GitHub] incubator-guacamole-server pull request #20: GUACAMOLE-87: Bump guacamole-se...

2016-08-25 Thread jmuehlner
GitHub user jmuehlner opened a pull request: https://github.com/apache/incubator-guacamole-server/pull/20 GUACAMOLE-87: Bump guacamole-server version numbers to 0.9.10-incubating. The Dockerfile is untouched because it needs more work before the release; simply bumping the version

[GitHub] incubator-guacamole-client pull request #59: GUACAMOLE-80 Remove trailing co...

2016-08-16 Thread jmuehlner
GitHub user jmuehlner opened a pull request: https://github.com/apache/incubator-guacamole-client/pull/59 GUACAMOLE-80 Remove trailing comma to fix sharing profile creation in mysql auth provider. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-guacamole-client pull request #58: GUACAMOLE-73: Clear permission ...

2016-08-16 Thread jmuehlner
GitHub user jmuehlner opened a pull request: https://github.com/apache/incubator-guacamole-client/pull/58 GUACAMOLE-73: Clear permission cache when saving or updating to force permission reload. The permissions live in the user cache, so clearing the user cache upon save/update

[GitHub] incubator-guacamole-server pull request #18: GUACAMOLE-51: Separate RDP keyb...

2016-08-16 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-server/pull/18#discussion_r75054973 --- Diff: src/protocols/rdp/keyboard.c --- @@ -0,0 +1,283 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

[GitHub] incubator-guacamole-client pull request #57: GUACAMOLE-79: LDAPConnection ha...

2016-08-14 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/57#discussion_r74715340 --- Diff: extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/user/UserService.java --- @@ -86,14 +87,18

[GitHub] incubator-guacamole-client pull request #57: GUACAMOLE-79: LDAPConnection ha...

2016-08-14 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/57#discussion_r74715309 --- Diff: extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/LDAPGuacamoleProperties.java --- @@ -145,4

[GitHub] incubator-guacamole-client pull request #57: GUACAMOLE-79: LDAPConnection ha...

2016-08-14 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/57#discussion_r74715279 --- Diff: extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/ConfigurationService.java --- @@ -206,4 +206,22

[GitHub] incubator-guacamole-client pull request #57: GUACAMOLE-79: LDAPConnection ha...

2016-08-14 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/57#discussion_r74715220 --- Diff: extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/ConfigurationService.java --- @@ -206,4 +206,22

[GitHub] incubator-guacamole-client pull request #52: GUACAMOLE-5: Implement administ...

2016-08-10 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/52#discussion_r74366237 --- Diff: guacamole/src/main/webapp/app/manage/controllers/manageSharingProfileController.js --- @@ -0,0 +1,406

[GitHub] incubator-guacamole-client pull request #47: GUACAMOLE-5: Add support for ma...

2016-08-04 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/47#discussion_r73638190 --- Diff: guacamole/src/main/webapp/app/manage/controllers/manageUserController.js --- @@ -918,6 +961,28 @@ angular.module('manage

[GitHub] incubator-guacamole-client pull request #47: GUACAMOLE-5: Add support for ma...

2016-08-04 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/47#discussion_r73637665 --- Diff: guacamole/src/main/webapp/app/manage/controllers/manageUserController.js --- @@ -918,6 +961,28 @@ angular.module('manage

[GitHub] incubator-guacamole-client pull request #46: GUACAMOLE-5: Implement user-fac...

2016-08-02 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/46#discussion_r73237046 --- Diff: guacamole/src/main/webapp/app/navigation/templates/guacMenu.html --- @@ -0,0 +1,7 @@ + +{{menuTitle

[GitHub] incubator-guacamole-client pull request #35: GUACAMOLE-5 Fix syntax error in...

2016-07-26 Thread jmuehlner
GitHub user jmuehlner opened a pull request: https://github.com/apache/incubator-guacamole-client/pull/35 GUACAMOLE-5 Fix syntax error in mysql create script by adding missing comma. We need to fix this to restore the mysql schema create script to working order. You can merge

[GitHub] incubator-guacamole-client pull request #33: GUACAMOLE-5: Invalidate share k...

2016-07-25 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/33#discussion_r72099720 --- Diff: extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc/tunnel

[GitHub] incubator-guacamole-server pull request #17: GUACAMOLE-5: Add per-user "read...

2016-07-24 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-server/pull/17#discussion_r72010270 --- Diff: src/protocols/rdp/user.c --- @@ -88,12 +94,24 @@ int guac_rdp_user_join_handler(guac_user* user, int argc, char** argv

[GitHub] incubator-guacamole-client pull request #32: GUACAMOLE-5: Implement connecti...

2016-07-21 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/32#discussion_r71819635 --- Diff: extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc/sharing

[GitHub] incubator-guacamole-client pull request #31: GUACAMOLE-5: Implement sharing ...

2016-07-19 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/31#discussion_r71465971 --- Diff: extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-postgresql/src/main/resources/org/apache/guacamole/auth/jdbc

[GitHub] incubator-guacamole-client pull request #31: GUACAMOLE-5: Implement sharing ...

2016-07-19 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/31#discussion_r71465897 --- Diff: extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-postgresql/src/main/resources/org/apache/guacamole/auth/jdbc

[GitHub] incubator-guacamole-client pull request #30: GUACAMOLE-5: Add concept of sha...

2016-07-17 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/30#discussion_r71088295 --- Diff: extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc/connection

[GitHub] incubator-guacamole-client pull request #30: GUACAMOLE-5: Add concept of sha...

2016-07-17 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/30#discussion_r71088214 --- Diff: guacamole/src/main/java/org/apache/guacamole/tunnel/UserTunnel.java --- @@ -0,0 +1,120 @@ +/* + * Licensed

[GitHub] incubator-guacamole-client pull request #30: GUACAMOLE-5: Add concept of sha...

2016-07-17 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/30#discussion_r71088115 --- Diff: guacamole/src/main/java/org/apache/guacamole/rest/permission/PermissionSetResource.java --- @@ -205,6 +212,19 @@ else

[GitHub] incubator-guacamole-client pull request #26: GUACAMOLE-5: Refactor /api/data...

2016-07-13 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/26#discussion_r70560762 --- Diff: guacamole/src/main/java/org/apache/guacamole/rest/user/UserResource.java --- @@ -0,0 +1,195 @@ +/* + * Licensed

[GitHub] incubator-guacamole-client pull request #26: GUACAMOLE-5: Refactor /api/data...

2016-07-13 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/26#discussion_r70559396 --- Diff: guacamole/src/main/java/org/apache/guacamole/rest/directory/DirectoryResource.java --- @@ -0,0 +1,265

[GitHub] incubator-guacamole-client pull request #26: GUACAMOLE-5: Refactor /api/data...

2016-07-13 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/26#discussion_r70557739 --- Diff: guacamole/src/main/java/org/apache/guacamole/rest/activeconnection/ActiveConnectionResource.java --- @@ -0,0 +1,70

[GitHub] incubator-guacamole-client pull request #24: GUACAMOLE-55: Add interface-lev...

2016-06-28 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/24#discussion_r68854489 --- Diff: guacamole/src/main/webapp/app/clipboard/directives/guacClipboard.js --- @@ -0,0 +1,312 @@ +/* + * Licensed

[GitHub] incubator-guacamole-server pull request #16: GUACAMOLE-44: Refactor the RDPD...

2016-06-16 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-server/pull/16#discussion_r67436488 --- Diff: src/protocols/rdp/guac_rdpdr/rdpdr_printer.h --- @@ -25,44 +25,32 @@ #include "rdpdr_service.h"

[GitHub] incubator-guacamole-client pull request #16: GUACAMOLE-44: Implement uploads...

2016-06-06 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/16#discussion_r65923641 --- Diff: guacamole/src/main/java/org/apache/guacamole/tunnel/GuacamoleStreamException.java --- @@ -0,0 +1,61

[GitHub] incubator-guacamole-client pull request #16: GUACAMOLE-44: Implement uploads...

2016-06-06 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/16#discussion_r6592 --- Diff: guacamole/src/main/java/org/apache/guacamole/tunnel/InterceptedStream.java --- @@ -0,0 +1,161 @@ +/* + * Licensed

[GitHub] incubator-guacamole-client pull request #14: GUACAMOLE-44: Slice uploaded fi...

2016-06-02 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/14#discussion_r65634966 --- Diff: guacamole-common-js/src/main/webapp/modules/ArrayBufferWriter.js --- @@ -62,6 +62,20 @@ Guacamole.ArrayBufferWriter

[GitHub] incubator-guacamole-server pull request #12: GUACAMOLE-25: Add audio input s...

2016-06-01 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-server/pull/12#discussion_r65482888 --- Diff: src/protocols/rdp/rdp_disp.h --- @@ -102,7 +103,7 @@ void guac_rdp_disp_free(guac_rdp_disp* disp); * * @param

[GitHub] incubator-guacamole-client pull request #11: GUACAMOLE-41: Add LDAP section ...

2016-06-01 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/11#discussion_r65446708 --- Diff: guacamole-docker/README.md --- @@ -129,17 +126,17 @@ Deploying Aoache Guacamole with LDAP authentication -e

[GitHub] incubator-guacamole-client pull request: GUACAMOLE-41: Add LDAP section to r...

2016-05-31 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/11#discussion_r65219207 --- Diff: guacamole-docker/README.md --- @@ -115,6 +119,35 @@ The process for doing this via the `mysql` utility included with MySQL

[GitHub] incubator-guacamole-server pull request: GUACAMOLE-35: Ensure perf...

2016-05-23 Thread jmuehlner
Github user jmuehlner commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-server/pull/10#discussion_r64323904 --- Diff: src/protocols/rdp/rdp_settings.c --- @@ -925,9 +925,31 @@ void guac_rdp_push_settings(guac_rdp_settings* guac_settings