Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update xhr.js #2607

Closed
wants to merge 1 commit into from
Closed

Update xhr.js #2607

wants to merge 1 commit into from

Conversation

bugdanov
Copy link

xhr.responseText is undefined when xhr responseType is 'blob'

xhr.responseText is undefined when dataType is 'arraybuffer'
@jquerybot
Copy link

Thank you for your pull request. It looks like this may be your first contribution to a jQuery Foundation project, if so we need you to sign our Contributor License Agreement (CLA).

📝 Please visit http://contribute.jquery.org/CLA/ to sign.

After you signed, the PR is checked again automatically after a minute. If there's still an issue, please reply here to let us know.


If you've already signed our CLA, it's possible your git author information doesn't match your CLA signature (both your name and email have to match), for more information, check the status of your CLA check.

@gibson042
Copy link
Member

The preexisting typeof check already handles undefined... are you trying to address an exception? If so, in which clients? We'd need a unit test covering the case, and a fix that doesn't break in IE9 as this one does.

@markelog
Copy link
Member

markelog commented Oct 8, 2015

It would be great, if you would sign CLA, provided isolated example failure through http://jsfiddle.net/ or similar resource and added test to support your change

@dmethvin
Copy link
Member

Since the reporter is not responding and we already have the essence of this bug covered by the issue in #2498, I'm going to close it.

@dmethvin dmethvin closed this Oct 20, 2015
@lock lock bot locked as resolved and limited conversation to collaborators Jan 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

None yet

5 participants