GUACAMOLE-1320: Provide chunked file upload mechanism

guacamole/src/main/frontend/src/app/rest/services/tunnelService.js

@@ -54,6 +54,15 @@ angular.module('rest').factory('tunnelService', ['$injector',
      */
     var DOWNLOAD_CLEANUP_WAIT = 5000;
 
+    /**
+     * The maximum size a chunk may be during uploadToStream() in bytes.
+     * 
+     * @private
+     * @constant
+     * @type Number
+     */
+    const CHUNK_SIZE = 1024 * 1024 * 4;
+
     /**
      * Makes a request to the REST API to get the list of all tunnels
      * associated with in-progress connections, returning a promise that
@@ -301,25 +310,60 @@ angular.module('rest').factory('tunnelService', ['$injector',
                 + '/' + encodeURIComponent(sanitizeFilename(file.name))
                 + '?token=' + encodeURIComponent(authenticationService.getCurrentToken());
 
-        var xhr = new XMLHttpRequest();
+        /**
+         * Creates a chunk of the inputted file to be uploaded.
+         * 
+         * @param {Number} offset
+         *      The byte at which to begin the chunk. 
+         * 
+         * @return {File}
+         *      The file chunk created by this function.
+         */
+        const createChunk = (offset) => {
+            var chunkEnd = Math.min(offset + CHUNK_SIZE, file.size);
+            const chunk = file.slice(offset, chunkEnd);
+            return chunk;
+        };
 
-        // Invoke provided callback if upload tracking is supported
+        /**
+         * POSTs the inputted chunks and recursively calls uploadHandler()
+         * until the upload is complete.
+         * 
+         * @param {File} chunk
+         *      The chunk to be uploaded to the stream.
+         * 
+         * @param {Number} offset
+         *      The byte at which the inputted chunk begins.
+         */ 
+        const uploadChunk = (chunk, offset) => {
+            var xhr = new XMLHttpRequest();
+            xhr.open('POST', url, true);
+
+            // Invoke provided callback if upload tracking is supported.
             if (progressCallback && xhr.upload) {
                 xhr.upload.addEventListener('progress', function updateProgress(e) {
-                progressCallback(e.loaded);
+                    progressCallback(e.loaded + offset);
                 });
-        }
+            };
 
-        // Resolve/reject promise once upload has stopped
+            // Continue to next chunk, resolve, or reject promise as appropriate
+            // once upload has stopped
             xhr.onreadystatechange = function uploadStatusChanged() {
 
-            // Ignore state changes prior to completion
+                // Ignore state changes prior to completion.
                 if (xhr.readyState !== 4)
                     return;
 
-            // Resolve if HTTP status code indicates success
+                // Resolve if last chunk or begin next chunk if HTTP status
-            if (xhr.status >= 200 && xhr.status < 300)
+                // code indicates success.
+                if (xhr.status >= 200 && xhr.status < 300) {
+                    offset += CHUNK_SIZE;
+
+                    if (offset < file.size)
+                        uploadHandler(offset);
+                    else
                         deferred.resolve();
+                }
 
                 // Parse and reject with resulting JSON error
                 else if (xhr.getResponseHeader('Content-Type') === 'application/json')
@@ -344,8 +388,21 @@ angular.module('rest').factory('tunnelService', ['$injector',
             };
 
             // Perform upload
-        xhr.open('POST', url, true);
+            xhr.send(chunk);
-        xhr.send(file);
+
+        };
+
+        /**
+         * Handler for the upload process. 
+         * 
+         * @param {Number} offset
+         *      The byte at which to begin the chunk.
+         */
+        const uploadHandler = (offset) => {
+            uploadChunk(createChunk(offset), offset);
+        };
+
+        uploadHandler(0);
 
         return deferred.promise;
 

guacamole/src/main/java/org/apache/guacamole/tunnel/InputStreamInterceptingFilter.java

@@ -94,8 +94,7 @@ public class InputStreamInterceptingFilter
     /**
      * Reads the next chunk of data from the InputStream associated with an
      * intercepted stream, sending that data as a "blob" instruction over the
-     * GuacamoleTunnel associated with this filter. If the end of the
+     * GuacamoleTunnel associated with this filter.
-     * InputStream is reached, an "end" instruction will automatically be sent.
      *
      * @param stream
      *     The stream from which the next chunk of data should be read.
@@ -112,9 +111,8 @@ public class InputStreamInterceptingFilter
             // End stream if no more data
             if (length == -1) {
 
-                // Close stream, send end if the stream is still valid
+                // Close stream
-                if (closeInterceptedStream(stream))
+                closeInterceptedStream(stream);
-                    sendEnd(stream.getIndex());
 
                 return;