Bug Tracker

Opened 6 years ago

Closed 5 years ago

#5786 closed bug (fixed)

xhr.send should test if data property is undefined

Reported by: rschouten Owned by:
Priority: minor Milestone: 1.4
Component: ajax Version: 1.3.2
Keywords: Cc:
Blocked by: Blocking:

Description

When using the $.ajax method with type 'POST' or 'DELETE' (probably other types too) and the data property is undefined, the string "undefined" appears in the request's payload. Normally this does not matter but in certain circumstances,like working with 3rd party APIs, this can lead to errors.

If sending 'undefined' as a string is not intentional, the data property should be tested and only sent if it is defined.

For example, at line 3633 of file http://ajax.googleapis.com/ajax/libs/jquery/1.3.2/jquery.js the code could be changed to:

if (s.data === undefined) {

xhr.send();

} else {

xhr.send(s.data);

}

Attachments (1)

index.html (665 bytes) - added by rschouten 6 years ago.
HTML with ajax to perform an HTTP DELETE

Download all attachments as: .zip

Change History (2)

Changed 6 years ago by rschouten

HTML with ajax to perform an HTTP DELETE

comment:1 Changed 5 years ago by dmethvin

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

Fixed in 1.4.3

Note: See TracTickets for help on using tickets.