[GitHub] [commons-vfs] garydgregory commented on a change in pull request #80: sftp add possibility to configure whether exec is enabled

[GitBox <gi...@apache.org>]

garydgregory commented on a change in pull request #80: sftp add possibility to configure whether exec is enabled
URL: https://github.com/apache/commons-vfs/pull/80#discussion_r375335941
 
 

 ##########
 File path: commons-vfs2/src/main/java/org/apache/commons/vfs2/provider/sftp/SftpFileSystemConfigBuilder.java
 ##########
 @@ -268,6 +269,21 @@ public ConfigRepository getConfigRepository(final FileSystemOptions opts) {
     public boolean isLoadOpenSSHConfig(final FileSystemOptions opts) {
         return this.getBoolean(opts, LOAD_OPENSSH_CONFIG, Boolean.FALSE);
     }
+    
+    /**
+     * Returns {@link Boolean#TRUE} if exec should be tested and enabled.
 
 Review comment:
   The API returns a `boolean`, not a `Boolean`, so the Javadoc should refer to `true`, not `Boolean#TRUE}.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services