echu2013 commented on a change in pull request #547:
URL: https://github.com/apache/guacamole-client/pull/547#discussion_r453132263
##########
File path: guacamole/src/main/webapp/app/client/controllers/clientController.js
##########
@@ -44,6 +44,16 @@ angular.module('client').controller('clientController',
['$scope', '$routeParams
var requestService = $injector.get('requestService');
var tunnelService = $injector.get('tunnelService');
var userPageService = $injector.get('userPageService');
+
+ /**
+ * Counter and limiter for maximum auto-reconnect attempts that we
should
+ * automatically try to establish a connection when
CLIENT_AUTO_RECONNECT or
+ * TUNNEL_AUTO_RECONNECT error types occur.
+ *
+ * @type Number
+ */
+ var AUTO_RECONNECT_TRY = 0;
+ var AUTO_RECONNECT_MAXTRY = 4;
Review comment:
@mike-jumper @necouchman I am sorry but I don't understand why would be
need to modify guacamole's schema to achieve this..
Why can't we just use `guacamole_connection_parameter` which already holds
all kind of profile parameters under column named `parameter_name`???
What I've done so far is an addition of 3 parameters (autoreconnect-disable,
autoreconnect-maxretries, autoreconnect-wait-seconds) which I need to get in
`clientController.js` somehow...
----------------------------------------------------------------
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:
[email protected]
