Bug 246355

Summary: Rekonq does not saves parameters when using "Try Again" button
Product: [Applications] rekonq Reporter: Stas <ss.paranoid>
Component: generalAssignee: Andrea Diamantini <adjam7>
Severity: normal CC: a.vanloon, fxrh, giovanni, maniandram01, pano_90
Priority: NOR    
Version: 0.5.0   
Target Milestone: 0.7   
Platform: openSUSE   
OS: Linux   
Latest Commit: Version Fixed In:

Description Stas 2010-07-31 16:35:50 UTC
Version:           0.5.0 (using KDE 4.4.4) 
OS:                Linux

When there are problems with internet connection and page loading fails and shows "Try Again" button - after pressing "Try Again" url parameters (like PHPBB topic IDs) are lost.

Reproducible: Always

Steps to Reproduce:
1. Open any PHPBB page.
2. Shut down internet connection.
3. Open any page in new tab.
4. Make internet connection available again.
5. Press "Try Again" button.

Actual Results:  
You will see the url without paremeters and message about wrong page or something like that.

Expected Results:  
The real page will be loaded and all parameters preserved.

OS: Linux (x86_64) release 2.6.34-12-default
Compiler: gcc
Comment 1 Panagiotis Papadopoulos 2010-08-01 11:38:07 UTC
I can confirm this too with the latest rekonq mainline.
Comment 2 Panagiotis Papadopoulos 2010-10-20 13:01:24 UTC
*** Bug 254722 has been marked as a duplicate of this bug. ***
Comment 3 Panagiotis Papadopoulos 2010-10-20 22:16:10 UTC
found the culprit for this bug:

Line 598 in webpage.cpp

QString urlString = Qt::escape(reply->url().toString(QUrl::RemoveUserInfo | QUrl::RemoveQuery | QUrl::RemovePath));
Comment 4 Giovanni Venturi 2010-11-21 15:08:14 UTC
Cannot backport to reKonq 0.6.x ?
Comment 5 Andrea Diamantini 2011-01-09 17:39:45 UTC
(In reply to comment #4)
> Cannot backport to reKonq 0.6.x ?

Yes, as soon as we fixed it.
Comment 6 Felix Rohrbach 2011-01-22 15:05:00 UTC
We need to cut the url because of a security issue, so it's not as easy as it looks like. We still try to fix that though.
Comment 7 Andrea Diamantini 2011-01-24 22:44:59 UTC
Git commit 1d83ce109628cf28269a849abec9786b9e920c39 by Andrea Diamantini.
Pushed by adjam into branch 'master'.

This evening I started working again on this bug. I did all the tests against XSS
and it seems ok. Hope it's true.


M  +7    -4    src/webpage.cpp     

Comment 8 Andrea Diamantini 2011-02-13 23:42:34 UTC
It seems working here now. Can someone please confirm it, so that we can close this bug (and backport to 0.6 as requested by Giovanni)?
Comment 9 Alexander van Loon 2011-02-18 19:50:30 UTC
I just updated to a more recent version of rekonq through this – https://launchpad.net/~yoann-laissus/+archive/rekonq-ppa – repository, which contains a daily build of 10 Februari. To try to reproduce the bug I followed the steps indicated by the bug reporter as closely as possible, I visited the phpBB forum – http://www.phpbb.com/community/index.php – of the phpBB community itself. I can confirm the bug did not occur.
Comment 10 Andrea Diamantini 2011-02-19 10:40:57 UTC
Backporting today...