[GitHub] incubator-guacamole-client pull request #122: GUACAMOLE-197: Implement Suppo...

2017-07-16 Thread necouchman
Github user necouchman commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/122#discussion_r127605470 --- Diff: extensions/guacamole-auth-radius/src/main/java/org/apache/guacamole/auth/radius/RadiusGuacamoleProperties.java --- @@ -0,0

[GitHub] incubator-guacamole-client pull request #122: GUACAMOLE-197: Implement Suppo...

2017-07-16 Thread necouchman
Github user necouchman commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/122#discussion_r127605481 --- Diff: extensions/guacamole-auth-radius/src/main/java/org/apache/guacamole/auth/radius/RadiusGuacamoleProperties.java --- @@ -0,0

[GitHub] incubator-guacamole-server pull request #105: GUACAMOLE-344: switching to up...

2017-07-16 Thread mike-jumper
Github user mike-jumper commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-server/pull/105#discussion_r127611769 --- Diff: Dockerfile --- @@ -77,6 +76,12 @@ COPY . "$BUILD_DIR" # Build guacamole-server from local source RUN yum -y

[GitHub] incubator-guacamole-server pull request #105: GUACAMOLE-344: switching to up...

2017-07-16 Thread mike-jumper
Github user mike-jumper commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-server/pull/105#discussion_r127611740 --- Diff: Dockerfile --- @@ -77,6 +76,12 @@ COPY . "$BUILD_DIR" # Build guacamole-server from local source RUN yum -y

[GitHub] incubator-guacamole-client pull request #122: GUACAMOLE-197: Implement Suppo...

2017-07-16 Thread mike-jumper
Github user mike-jumper commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/122#discussion_r127611971 --- Diff: extensions/guacamole-auth-radius/src/main/java/org/apache/guacamole/auth/radius/RadiusConnectionService.java --- @@ -0,0

[GitHub] incubator-guacamole-client pull request #122: GUACAMOLE-197: Implement Suppo...

2017-07-16 Thread necouchman
Github user necouchman commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/122#discussion_r127613506 --- Diff: extensions/guacamole-auth-radius/src/main/java/org/apache/guacamole/auth/radius/RadiusConnectionService.java --- @@ -0,0 +

[GitHub] incubator-guacamole-client pull request #122: GUACAMOLE-197: Implement Suppo...

2017-07-16 Thread necouchman
Github user necouchman commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/122#discussion_r127613709 --- Diff: extensions/guacamole-auth-radius/src/main/java/org/apache/guacamole/auth/radius/RadiusConnectionService.java --- @@ -0,0 +

[GitHub] incubator-guacamole-client pull request #122: GUACAMOLE-197: Implement Suppo...

2017-07-16 Thread necouchman
Github user necouchman commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/122#discussion_r127614753 --- Diff: extensions/guacamole-auth-radius/src/main/java/org/apache/guacamole/auth/radius/RadiusConnectionService.java --- @@ -0,0 +

[GitHub] incubator-guacamole-client pull request #122: GUACAMOLE-197: Implement Suppo...

2017-07-16 Thread necouchman
Github user necouchman commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/122#discussion_r127614757 --- Diff: extensions/guacamole-auth-radius/src/main/java/org/apache/guacamole/auth/radius/RadiusConnectionService.java --- @@ -0,0 +

[GitHub] incubator-guacamole-client pull request #177: GUACAMOLE-339: Add Remote Host...

2017-07-16 Thread necouchman
GitHub user necouchman opened a pull request: https://github.com/apache/incubator-guacamole-client/pull/177 GUACAMOLE-339: Add Remote Host to Connection History Table This pull request adds the necessary fields and methods to the JDBC module to track the remote host in the connectio

[GitHub] incubator-guacamole-client pull request #177: GUACAMOLE-339: Add Remote Host...

2017-07-16 Thread mike-jumper
Github user mike-jumper commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/177#discussion_r127626203 --- Diff: extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-mysql/schema/001-create-schema.sql --- @@ -322,6 +322,7 @@ CRE

[GitHub] incubator-guacamole-client pull request #177: GUACAMOLE-339: Add Remote Host...

2017-07-16 Thread necouchman
Github user necouchman commented on a diff in the pull request: https://github.com/apache/incubator-guacamole-client/pull/177#discussion_r127626364 --- Diff: extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-mysql/schema/001-create-schema.sql --- @@ -322,6 +322,7 @@ CREA

Re: [DISCUSS] Scope of 0.9.13-incubating

2017-07-16 Thread Kaushik Srinivasan
Hi All, I have tried using the guacamole-auth-cas with 0.9.13 as suggested. I further used jdbc connection (mysql) for connection properties. The CAS authentication works now but does not redirect to my connection after that. I verified that the jdbc connection works independently ( by removing a

[GitHub] incubator-guacamole-client pull request #177: GUACAMOLE-339: Add Remote Host...

2017-07-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-guacamole-client/pull/177 --- 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

Re: [DISCUSS] Scope of 0.9.13-incubating

2017-07-16 Thread Mike Jumper
On Sun, Jul 16, 2017 at 7:49 PM, Kaushik Srinivasan wrote: > Hi All, > > I have tried using the guacamole-auth-cas with 0.9.13 as suggested. I > further used jdbc connection (mysql) for connection properties. > > The CAS authentication works now Excellent. > but does not redirect to my connec

Re: [DISCUSS] Scope of 0.9.13-incubating

2017-07-16 Thread Kaushik Srinivasan
Hi Mike, Regarding redirection after CAS authentication, just to test I removed the jdbc extension and launched the application. After CAS authentication the same error (as I've previously shown) is thrown which makes me think there's something up with CAS module. Yes, I'm using 0.9.13 for everyt

Re: [DISCUSS] Scope of 0.9.13-incubating

2017-07-16 Thread Nicholas Couchman
I will take a look at the CAS module tomorrow and see if I can reproduce the error you're getting in the log file. However, keep in mind the following things: - As Mike pointed out, the auto-redirect to a connection only happens if a user has 1 and only 1 connection available. If there is mor

Re: [DISCUSS] Scope of 0.9.13-incubating

2017-07-16 Thread Kaushik Srinivasan
Thanks a lot Nick. I'm just stating few points to make sure I'm not doing something silly. There is only one connection configured in the mysql table which works perfectly fine standalone. When there are no connections available and I'm using CAS in conjunction with the jdbc the the error I said