Bug Tracker

Opened 6 years ago

Closed 5 years ago

#7248 closed enhancement (wontfix)

XHR response Blob support

Reported by: zealot0630@… Owned by:
Priority: low Milestone:
Component: ajax Version: 1.4.3
Keywords: needsreview, ajaxrewrite Cc:
Blocked by: Blocking:

Description

Currently only text and XML is supported in jQuery, but XHR2 introduced a new blob response (​http://www.w3.org/TR/XMLHttpRequest2/#the-responseblob-attribute). I think it is worth considering.

Change History (6)

comment:1 Changed 6 years ago by Zealot

patch 

diff --git a/jquery-1.4.2.js b/jquery-1.4.2.js
index 0c8b750..c7124b3 100644
--- a/jquery-1.4.2.js
+++ b/jquery-1.4.2.js
@@ -5346,7 +5346,7 @@ jQuery.extend({
 	httpData: function( xhr, type, s ) {
 		var ct = xhr.getResponseHeader("content-type") || "",
 			xml = type === "xml" || !type && ct.indexOf("xml") >= 0,
-			data = xml ? xhr.responseXML : xhr.responseText;
+			data = type === "blob" ? { type: ct, blob: xhr.responseBlob } : xml ? xhr.responseXML : xhr.responseText;
 
 		if ( xml && data.documentElement.nodeName === "parsererror" ) {
 			jQuery.error( "parsererror" );

comment:2 Changed 6 years ago by snover

  • Keywords needsreview added

Bikeshed/feature creep. Marking for review.

comment:3 Changed 6 years ago by rwaldron

  • Component changed from unfiled to ajax
  • Priority changed from undecided to low

comment:4 Changed 6 years ago by paul.irish

There are a lot of things in XHR2 that would be useful. Probably good for a whole plugin to dump all sorts of usefulness onto $.ajax.

comment:5 Changed 6 years ago by rwaldron

  • Keywords ajaxrewrite added

comment:6 Changed 5 years ago by john

  • Resolution set to wontfix
  • Status changed from new to closed

Yeah, let's push this off to a plugin (since Ajax is extensible now) - and we can reintegrate it later.

Note: See TracTickets for help on using tickets.