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

    https://github.com/apache/guacamole-client/pull/353#discussion_r245858704
  
    --- Diff: 
guacamole-ext/src/main/java/org/apache/guacamole/net/auth/simple/SimpleConnection.java
 ---
    @@ -68,21 +72,34 @@ public SimpleConnection() {
          * @param identifier The identifier to associate with this connection.
          * @param config The configuration describing how to connect to this
          *               connection.
    +     * 
    +     * @throws GuacamoleException
    +     *     If the default proxy configuration cannot be retrieved.
          */
         public SimpleConnection(String name, String identifier,
    -            GuacamoleConfiguration config) {
    +            GuacamoleConfiguration config) throws GuacamoleException {
             
             // Set name
    -        setName(name);
    +        super.setName(name);
     
             // Set identifier
    -        setIdentifier(identifier);
    +        super.setIdentifier(identifier);
     
             // Set config
    -        setConfiguration(config);
    +        super.setConfiguration(config);
             this.config = config;
    +        this.proxyConfig = new 
LocalEnvironment().getDefaultGuacamoleProxyConfiguration();
     
         }
    +    
    +    public SimpleConnection(String name, String identifier,
    --- End diff --
    
    Documented.


---

Reply via email to