bugGNU Wget - Bugs: bug #33149, --convert-links does not escape ?...

 
 

bug #33149: --convert-links does not escape ? character in links

Submitter:  tot-to <totto>
Submitted:  Sat 23 Apr 2011 10:13:41 PM UTC
   
 
Category:  Program Logic Severity:  3 - Normal
Priority:  5 - Normal Status:  None
Privacy:  Public Assigned to:  None
Originator Name:  Open/Closed:  Open
Release:  1.12 Operating System:  GNU/Linux
Reproducibility:  Every Time Fixed Release:  None
Planned Release:  None Regression:  None
Work Required:  None Patch Included:  No
* Mandatory Fields

Add a New Comment Rich Markup
   

Wed 28 Aug 2013 01:15:10 AM UTC, comment #2: 

This issue is still present in wget 1.141.  Thanks to the original submitter for the workaround suggestion.

Alex Lancaster <alexl>
Mon 25 Apr 2011 09:47:47 AM UTC, comment #1: 

I'm joining this issue. I think that behaviour should be consistent with no dependency on --restrict-file-names.
I think it compares link and file name and forgets about special characters. So not doing rewrite at all.
Because when it actually do rewriting of links it replaces special chars. For example with -E.

Alexander Zubkov <zubkov>
Sat 23 Apr 2011 10:13:41 PM UTC, original submission:  

When I download some site with wget:
wget -r -k http:/mysite.com/
The wget generates file like this:
page.php?param1=value1&param2=value2
and changes links in file to appropriate. But browsers suppose the symbols after ? are GET-paramerter, but not a part of file.
If open file with ? in Firefox by open dialog, we will see %3F in address instead of ? Therefore, I believe that wget -k must change ? to %3F in html-code.
The workaround is using option --restrict-file-names=windows. Because ? is not acceptable filename character in this operating system, it will be changed to @ both in filenames and links.

tot-to <totto>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by alexl (Posted a comment)
  • -email is unavailable- added by zubkov (Posted a comment)
  • -email is unavailable- added by zubkov
  • -email is unavailable- added by totto (Submitted the item)
  •  

    Follows 1 latest change.

    Date Changed by Updated Field Previous Value => Replaced by
    2011-04-25 zubkov Carbon-Copy- Added zubkov

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code