Bug 17631

Summary: URL encoding problem with spaces
Product: WebKit Reporter: Ismail Donmez <ismail>
Component: WebCore Misc.Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, jchaffraix, michelangelo
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.5   
Attachments:
Description Flags
reduced test case none

Ismail Donmez
Reported 2008-03-01 16:56:33 PST
Attached is a web archive of http://www.milliyet.com.tr , one of the most popular newspaper in Turkey. To reproduce the bug click on the upper right picture (which is the picture of our president), the one with moustache. Which also contains the text "Başbakanlık'ta ..." Now if you click the link its supposed to go to default.aspx?...blabla but instead it goes to %20%20...%20default.aspx?...blabla , somehow the whitespace is getting encoded.
Attachments
reduced test case (512 bytes, text/html)
2008-03-03 13:54 PST, Alexey Proskuryakov 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Ismail Donmez
Comment 1 2008-03-01 17:03:12 PST
Uploaded web archive to http://cekirdek.pardus.org.tr/~ismail/tmp/milliyet.webarchive , too big to attach.
Alexey Proskuryakov
Comment 2 2008-03-03 13:54:20 PST
Created attachment 19502 [details] reduced test case
Alexey Proskuryakov
Comment 3 2008-03-03 13:56:47 PST
Confirmed with r30710. Firefox works as expected; haven't tested IE.
Ismail Donmez
Comment 4 2008-03-03 15:00:01 PST
Maksim Orlovich of KHTML fame fixed this for khtml: http://lists.kde.org/?l=kde-commits&m=120458484225762&w=2
Ismail Donmez
Comment 5 2008-10-21 09:08:17 PDT
This bug is still reproducable with latest ToT and makes one of the biggest newspaper website (in Turkey) unusable.
Alexey Proskuryakov
Comment 6 2008-11-20 05:49:45 PST
<rdar://problem/6388381>
Ismail Donmez
Comment 7 2010-07-08 07:37:35 PDT
Test passes with latest trunk, closing as fixed.
Alexey Proskuryakov
Comment 8 2010-07-08 10:30:16 PDT
Likely fixed in r58844. The test case is a little different, so I landed it in r62800.
Note You need to log in before you can comment on or make changes to this bug.