Side navigation
#6908 closed bug (fixed)
Opened August 16, 2010 01:48PM UTC
Closed October 25, 2010 08:16PM UTC
Last modified October 27, 2010 12:27AM UTC
$.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.
Attachments (0)
Change History (7)
Changed October 21, 2010 02:06AM UTC by comment:1
milestone: | 1.4.3 → 1.4.4 |
---|---|
priority: | → undecided |
status: | new → open |
Changed October 21, 2010 02:07AM UTC by comment:2
priority: | undecided → low |
---|
Changed October 21, 2010 04:13AM UTC by comment:3
keywords: | → ajax, case-senstivity |
---|---|
owner: | → SlexAxton |
status: | open → assigned |
Changed October 21, 2010 04:16AM UTC by comment:4
I have made a pull request for this. http://github.com/jquery/jquery/pull/62
Changed October 21, 2010 10:57PM UTC by comment:5
milestone: | 1.4.4 → 1.4.5 |
---|
Retargeting due to severity of regressions in 1.4.3.
Changed October 25, 2010 08:16PM UTC by comment:6
milestone: | 1.4.5 → 1.4.4 |
---|---|
resolution: | → fixed |
status: | assigned → closed |
version: | 1.4.2 → 1.4.3 |
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?