Uploaded image for project: 'Guacamole'
  1. Guacamole
  2. GUACAMOLE-327

File upload/download issues in IE

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Duplicate
    • 0.9.12-incubating
    • None
    • guacamole-client
    • None

    Description

      Internet Explorer, including 11, doesn't have support for location.origin so when trying to build the URL for the XMLHttpRequests in tunnelService.downloadStream and tunnelService.uploadToStream, the string "undefined" gets prepended onto the URL, rendering that functionality unusable.

      The workaround/polyfill for this is pretty simple and well documented on the web and is basically:

      if (!window.location.origin) {
        window.location.origin = window.location.protocol + '//' + window.location.hostname + (window.location.port ? (':' + window.location.port) : '');
      }
      

      reference: https://gist.github.com/hbogs/7908703

      Additionally, IE seems to have a problem with the fallback code for removing the download iframe from the body. If the iframe is successfully removed by the callback of the iframe.onload handler, the stream.onend callback encounters an error because the iframe is no longer attached. A check for iframe's parentElement being defined is a workaround:

      stream.onend = function downloadComplete() {
                  $window.setTimeout(function cleanupIframe() {
                      if (iframe.parentElement) {
                          document.body.removeChild(iframe);
                      }
                  }, DOWNLOAD_CLEANUP_WAIT);
              };
      

      Finally, IE 10 and IE 11 are not able to download filetypes the browser will automatically try to open (like .txt and .jpg files). The download appears to just not do anything. The same remote system is able to download other extensions like .exe and .zip download just fine. This is true for either method (the menu file browser or the guacctl utility).

      Attachments

        Issue Links

          Activity

            People

              mjumper Mike Jumper
              shauntarves Shaun Tarves
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: