Github user necouchman commented on a diff in the pull request:
https://github.com/apache/incubator-guacamole-server/pull/93#discussion_r123893854
--- Diff: src/protocols/rdp/rdp_settings.h ---
@@ -359,6 +359,14 @@ typedef struct guac_rdp_settings {
* the destination directory is otherwise ambiguous).
*/
char* sftp_directory;
+
+ /**
+ * The interval at which SSH keepalive messages are sent to the server
for
+ * SFTP connections. The default is 0 (disabling keepalives), and a
value
+ * of 1 is automatically increased to 2 by libssh2 to avoid busy loop
corner
+ * cases.
+ */
+ unsigned sftp_server_alive_interval;
--- End diff --
How does it look, now? It isn't the cleanest, because 0 is actually valid
(to disable keepalive). So, I'm setting negative values to 0 (since negative
values don't make sense), and then just warning that 1 gets rounded up to 2 and
allowing libssh2 to do that internally.
---
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 the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
