Github user necouchman commented on a diff in the pull request:

    https://github.com/apache/guacamole-client/pull/319#discussion_r220506258
  
    --- Diff: 
extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-mysql/src/main/resources/org/apache/guacamole/auth/jdbc/connection/ConnectionRecordMapper.xml
 ---
    @@ -79,7 +79,10 @@
                 #{record.sharingProfileIdentifier,jdbcType=VARCHAR},
                 #{record.sharingProfileName,jdbcType=VARCHAR},
                 (SELECT user_id FROM guacamole_user
    -             WHERE username = #{record.username,jdbcType=VARCHAR}),
    +             JOIN guacamole_entity ON guacamole_user.entity_id = 
guacamole_entity.entity_id
    --- End diff --
    
    Oops - I think I was confusing that block of SQL code for the `INSERT` 
statement with a query that would `SELECT` rows from the table.  My bad.


---

Reply via email to