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

 
 

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

Submitter:  Orion Poplawski <opoplawski>
Submitted:  Thu 09 Oct 2014 09:34:38 PM UTC
   
 
Category:  Program Logic Severity:  3 - Normal
Priority:  3 - Low Status:  Wont Fix
Privacy:  Public Assigned to:  None
Originator Name:  Open/Closed:  Closed
Release:  trunk Operating System:  GNU/Linux
Reproducibility:  Every Time Fixed Release:  None
Planned Release:  None Regression:  None
Work Required:  0 - Hours Patch Included:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

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>
Group 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>
Group 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>
Group 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 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 psdh (Posted a comment)
  • -email is unavailable- added by darnir (Posted a comment)
  • -email is unavailable- added by rockdaboot (Posted a comment)
  • -email is unavailable- added by opoplawski (Submitted the item)
  •  

    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2015-03-06 darnir StatusConfirmed Wont Fix
        Open/ClosedOpen Closed
    2014-11-18 darnir Priority5 - Normal 3 - Low
    2014-10-17 rockdaboot StatusNone Confirmed
        Work RequiredNone 0 - Hours

    Back to the top

    Powered by Savane 3.13-cf05.
    Corresponding source code