Fix tar-gz reading of Dockerfiles

- Fixes TAR to actually use data
- Fixes buffers for Gunzip to be a proper ArrayBufferView

Fixes #1832

static/js/services/datafile-service.js

@@ -45,7 +45,7 @@ angular.module('quay').factory('DataFileService', [function() {
   };
 
   dataFileService.tryAsTarGz_ = function(buf, success, failure) {
-    var gunzip = new Zlib.Gunzip(buf);
+    var gunzip = new Zlib.Gunzip(new Uint8Array(buf));
     var plain = null;
 
     try {
@@ -56,8 +56,7 @@ angular.module('quay').factory('DataFileService', [function() {
     }
 
     if (plain.byteLength == 0) {
-      success([]);
+      plain = buf;
-      return;
     }
 
     dataFileService.tryAsTar_(plain, success, failure);
@@ -77,7 +76,7 @@ angular.module('quay').factory('DataFileService', [function() {
       return parts.join('/');
     };
 
-    var handler = new Untar(buf);
+    var handler = new Untar(new Uint8Array(buf));
     handler.process(function(status, read, files, err) {
       switch (status) {
         case 'error':

static/js/tar.js

@@ -371,7 +371,7 @@ if (!Array.prototype.some) {
      * @param data The data, in Uint8Array form.
      */
     function Untar(data) {
-        this.data = [];
+        this.data = data;
     }
 
     Untar.prototype.process = function(cb, opt_filter) {