Bug Tracker

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#6908 closed bug (fixed)

$.ajax doesn't ignore case of scheme and host in URL.

Reported by: YOCKOW Owned by: SlexAxton
Priority: low Milestone: 1.4.4
Component: ajax Version: 1.4.3
Keywords: ajax, case-senstivity Cc:
Blocked by: Blocking:

Description

(ver 1.4.2)

line 5058: remote = parts && (parts[1] && parts[1] !== location.protocol
parts[2] !== location.host);

SHOULD BE

remote = parts && (parts[1] && parts[1].toLowerCase() !== location.protocol.toLowerCase()
parts[2].toLowerCase() !== location.host.toLowerCase());

because scheme in URL and domain names are case-insensitive.

Change History (7)

comment:1 Changed 7 years ago by SlexAxton

Milestone: 1.4.31.4.4
Priority: undecided
Status: newopen

I think this is valid. I'm not sure this comes up a lot, but if we think it's worth the few extra bytes, it could save someone some pain. Thoughts?

comment:2 Changed 7 years ago by SlexAxton

Priority: undecidedlow

comment:3 Changed 7 years ago by SlexAxton

Keywords: ajax case-senstivity added
Owner: set to SlexAxton
Status: openassigned

comment:4 Changed 7 years ago by SlexAxton

I have made a pull request for this. http://github.com/jquery/jquery/pull/62

comment:5 Changed 7 years ago by snover

Milestone: 1.4.41.4.5

Retargeting due to severity of regressions in 1.4.3.

comment:6 Changed 7 years ago by john

Milestone: 1.4.51.4.4
Resolution: fixed
Status: assignedclosed
Version: 1.4.21.4.3

comment:7 Changed 7 years ago by snover

#6873 is a duplicate of this ticket.

Note: See TracTickets for help on using tickets.