bugGNU Wget - Bugs: bug #43394, no_parent does not handle paths...

 
 

bug #43394: no_parent does not handle paths with duplicate /'s.

Submitted by:  Orion Poplawski <opoplawski>
Submitted on:  Thu 09 Oct 2014 09:34:38 PM UTC  
 
Category: Program LogicSeverity: 3 - Normal
Priority: 3 - LowStatus: Wont Fix
Privacy: PublicAssigned to: None
Originator Name: Open/Closed: Closed
Release: trunkOperating System: GNU/Linux
Reproducibility: Every TimeFixed Release: None
Planned Release: NoneRegression: None
Work Required: 0 - HoursPatch Included: None

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Fri 06 Mar 2015 12:41:33 PM UTC, comment #5:

Hi!
after irc discussions with darnir, and searching through links in the comments and the RFC. It seems like the protocol does not recommend the elimination of the duplicate slashes.

This implies that there would be no requirement of any code change and hence this bug can be closed.

Prabhjyot <psdh>
Tue 18 Nov 2014 07:31:53 AM UTC, comment #4:

While Wikipedia states that as part of Path Normalization it is valid to eliminate duplicate slashes, the referenced RFC, RFC 3986 doesn't seem to mention anything along these lines.

However, as part of searching for this, I happened across the following StackExchange answer:
http://webmasters.stackexchange.com/a/8381

Which claims that such a behaviour is indeed against the letter of the RFCs. Yet, upon closer investigation, it seems like Wget SHOULD issue a request for //SUM17/D617249504/S00000

This is a parser error, and I guess it ought to be fixed

Darshit Shah <darnir>
Project Administrator
Tue 18 Nov 2014 07:10:00 AM UTC, comment #3:

Is there any documentation that says clients must handle such path names?

It seems to me that the server is just broken, and a simple email to the webmaster should fix it.

Darshit Shah <darnir>
Project Administrator
Mon 13 Oct 2014 03:35:57 PM UTC, comment #2:

wget -r -nH --cut-dirs=1 -np -e robots=off --debug http://jsoc.stanford.edu/SUM17/D617249504/S00000

wget-1.14-12.fc20.x86_64

GNU Wget 1.14 built on linux-gnu.

+digest +https +ipv6 +iri +large-file +nls +ntlm +opie +ssl/openssl

Wgetrc:
/etc/wgetrc (system)
Locale: /usr/share/locale
Compile: gcc -DHAVE_CONFIG_H -DSYSTEM_WGETRC="/etc/wgetrc"
-DLOCALEDIR="/usr/share/locale" -I. -I../lib -I../lib -O2 -g -pipe
-Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector-strong
--param=ssp-buffer-size=4 -grecord-gcc-switches -m64 -mtune=generic
Link: gcc -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions
-fstack-protector-strong --param=ssp-buffer-size=4
-grecord-gcc-switches -m64 -mtune=generic -lssl -lcrypto
/usr/lib64/libssl.so /usr/lib64/libcrypto.so /usr/lib64/libz.so
-ldl -lz -lz -lidn -luuid -lpcre ftp-opie.o openssl.o http-ntlm.o
../lib/libgnu.a

Orion Poplawski <opoplawski>
Mon 13 Oct 2014 09:01:44 AM UTC, comment #1:

Could you give us the complete command line you are using, please.
Also, the Wget version would be of interest (wget --version).

Tim Ruehsen <rockdaboot>
Project Administrator
Thu 09 Oct 2014 09:34:38 PM UTC, original submission:

This page incorrectly has a double slash after the host name. wget should discard any duplicate slashes.

Deciding whether to enqueue "http://jsoc.stanford.edu//SUM17/D617249504/S00000/mdi.vw_V_sht_gf_72d.3232.48.fits".
Going to "/SUM17/D617249504/S00000" would escape "SUM17/D617249504" with no_parent on.
Decided NOT to load it.

Not sure where this parsing should go though.

Orion Poplawski <opoplawski>

 

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

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by psdh (Posted a comment)
  • -unavailable- added by darnir (Posted a comment)
  • -unavailable- added by rockdaboot (Posted a comment)
  • -unavailable- added by opoplawski (Submitted the item)
  •  

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 06 Mar 2015 03:44:23 PM UTCdarnirStatusConfirmed=>Wont Fix
      Open/ClosedOpen=>Closed
    Tue 18 Nov 2014 07:31:53 AM UTCdarnirPriority5 - Normal=>3 - Low
    Fri 17 Oct 2014 11:44:44 AM UTCrockdabootStatusNone=>Confirmed
      Work RequiredNone=>0 - Hours

    Back to the top


    Powered by Savane 3.1-cleanup1