Re: [PR] GUACAMOLE-2002: Allow connection clipboard limits to be configured [guacamole-server]

["eugen-keeper (via GitHub)" <gi...@apache.org>]

eugen-keeper commented on code in PR #564:
URL: https://github.com/apache/guacamole-server/pull/564#discussion_r1866250365


##########
src/protocols/kubernetes/settings.c:
##########
@@ -418,6 +425,19 @@ guac_kubernetes_settings* guac_kubernetes_parse_args(guac_user* user,
         guac_user_parse_args_int(user, GUAC_KUBERNETES_CLIENT_ARGS, argv,
                 IDX_BACKSPACE, GUAC_TERMINAL_DEFAULT_BACKSPACE);
 
+    /* Set the maximum number of bytes to allow within the clipboard. */
+    settings->clipboard_buffer_size =
+        guac_user_parse_args_int(user, GUAC_KUBERNETES_CLIENT_ARGS, argv,
+                IDX_CLIPBOARD_BUFFER_SIZE, 0);
+
+    /* Use default clipboard buffer size if given one is invalid. */            
+    if (settings->clipboard_buffer_size <= 0) {

Review Comment:
   Fixed. Added max/min clipboard size limits.



-- 
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: dev-unsubscribe@guacamole.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org