bugGNU Wget - Bugs: bug #31419, --convert-links doesn't work if...

 
 

bug #31419: --convert-links doesn't work if link contains utf-8 characters

Submitter:  None
Submitted:  Fri 22 Oct 2010 06:00:30 PM UTC
   
 
Category:  None Severity:  3 - Normal
Priority:  5 - Normal Status:  None
Privacy:  Public Assigned to:  None
Originator Name:  Manfred Koizar Originator Email:  -email is unavailable-
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
   

Sun 24 Oct 2010 12:43:22 AM UTC, comment #3: 

Attached one line change treats any difference between origurl and u->url like a redirection and seems to work for me.  Don't know whether this counts as a fix or just a workaround ...

(file #21779)

Anonymous
Fri 22 Oct 2010 06:11:43 PM UTC, comment #2: 

And here the description I lost on my first try:
The link in wgtest/index.html


Look <a href="http://www.Koizar.eu/wgtst/fürDich/test.html">here</a>!


should be converted to


Look <a href="fürDich/test.html">here</a>!


But it isn't. OTOH the backreference


Go <a href="http://www.Koizar.eu/wgtst/index.html">back</a>!


in wgtest/fürDich/test.html is successfully converted to


Go <a href="../index.html">back</a>!


I suspect the problem lies in or near convert_links_in_hashtable() in convert.c.

Anonymous
Fri 22 Oct 2010 06:05:04 PM UTC, comment #1: 

Second try, the test case:


wget  --recursive --level=inf --convert-links  http://www.Koizar.eu/wgtst/index.html


Anonymous
Fri 22 Oct 2010 06:00:30 PM UTC, original submission:  


Here is a test case for those who want to investigate closer:


wget  --recursive --level=inf --convert-links  http://www.Koizar.eu/wgtst/index.html

In wgtst/index.html this line

+verbatim+
Look <a href="http://www.Koizar.eu/wgtst/fürDich/test.html">here</a>!


should be converted to


Look <a href="fürDich/test.html">here</a>!


but it isn't.  The backreference in wgtst/fürDich/test.html


Go <a href="http://www.Koizar.eu/wgtst/index.html">back</a>!


works:


Go <a href="../index.html">back</a>!


I suspect the problem lies in or near convert_links_in_hashtable() in convert.c.

Anonymous

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #21779:  02_convert_link.diff added by None (537B - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

Follows 1 latest change.

Date Changed by Updated Field Previous Value => Replaced by
2010-10-24 None Attached File- Added 02_convert_link.diff, #21779

Back to the top

Powered by Savane 3.13-f8d8.
Corresponding source code