WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 17631
URL encoding problem with spaces
https://bugs.webkit.org/show_bug.cgi?id=17631
Summary
URL encoding problem with spaces
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.
Top of Page
Format For Printing
XML
Clone This Bug