necouchman commented on code in PR #1117:
URL: https://github.com/apache/guacamole-client/pull/1117#discussion_r2353438642
##########
guacamole/src/main/java/org/apache/guacamole/GuacamoleSession.java:
##########
@@ -61,6 +62,11 @@ public class GuacamoleSession {
*/
private final Map<String, UserTunnel> tunnels = new ConcurrentHashMap<>();
+ /**
+ * Creation times for all tunnels, indexed by tunnel UUID.
+ */
+ private final Map<String, Long> tunnelCreationTimes = new
ConcurrentHashMap<>();
+
Review Comment:
Thoughts on whether it makes sense to do it this way, or to just add a way
within the various `Tunnel` interfaces/classes, to track tunnel creation time?
The `GuacamoleTunnel` interface could get a new method `getCreationTime()` (or
whatever makes sense), and then one of the implementing classes could get
either a `Long` or perhaps one of the Java `Time`/`DateTime` fields to track
this?
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
