In url.c / url_file_name() an empty query is not used for the filename:
/* Append "?query" to the file name. */
u_query = u->query && *u->query ? u->query : NULL;
Should it be patched here ?
Mit freundlichen Grüßen
Tim Rühsen
Am Thursday 29 March 2012 schrieb Tim Ruehsen:
> Just some more infos:
> It is reproducible with the latest trunk version.
>
> The problem seems to be empty queries like in main.css (original):
> src: url('/TLBB/fbinir/mult/stagsans-book-webfont.eot');^M
> src: url('/TLBB/fbinir/mult/stagsans-book-webfont.eot?#iefix')
> format('embedded-opentype'),^M
>
> BTW, empty queries are absolutely legal (rfc 2396: query = *uric).
>
> The downloader downloads
> /TLBB/fbinir/mult/stagsans-book-webfont.eot
> and
> /TLBB/fbinir/mult/stagsans-book-webfont.eot?
> and saves them both into the same file
> Saving to: 'accionistaseinversores.bbva.com/TLBB/fbinir/mult/stagsans-book-
> webfont.eot'
>
> I assume the hashmaps 'dl_url_file_map' and 'dl_file_url_map' are now out
> of sync.
> Now the scanning can't find a local file for the first download and thus
> does not translate it to local name but to a complete name.
>
> Here is some debug output where you can see it (look for 'complete', which
> should be local):
>
> Scanning accionistaseinversores.bbva.com/TLBB/fbinir/css/main.css?v=1 (from
> http://accionistaseinversores.bbva.com/TLBB/fbinir/css/main.css?v=1)
> Loaded accionistaseinversores.bbva.com/TLBB/fbinir/css/main.css?v=1 (size
> 99449).
> accionistaseinversores.bbva.com/TLBB/fbinir/css/main.css?v=1:
> merge('http://accionistaseinversores.bbva.com/TLBB/fbinir/css/main.css?v=1'
> , '/TLBB/fbinir/mult/stagsans-book-webfont.eot') ->
> http://accionistaseinversores.bbva.com/TLBB/fbinir/mult/stagsans-book-
> webfont.eot
> appending
> 'http://accionistaseinversores.bbva.com/TLBB/fbinir/mult/stagsans-
> book-webfont.eot' to urlpos.
> Found URI: [url('/TLBB/fbinir/mult/stagsans-book-webfont.eot')] at 2404
> [/TLBB/fbinir/mult/stagsans-book-webfont.eot]
> accionistaseinversores.bbva.com/TLBB/fbinir/css/main.css?v=1:
> merge('http://accionistaseinversores.bbva.com/TLBB/fbinir/css/main.css?v=1'
> , '/TLBB/fbinir/mult/stagsans-book-webfont.eot?#iefix') ->
> http://accionistaseinversores.bbva.com/TLBB/fbinir/mult/stagsans-book-
> webfont.eot?#iefix
> appending
> 'http://accionistaseinversores.bbva.com/TLBB/fbinir/mult/stagsans-
> book-webfont.eot?' to urlpos.
> Found URI: [url('/TLBB/fbinir/mult/stagsans-book-webfont.eot?#iefix')] at
> 2462 [/TLBB/fbinir/mult/stagsans-book-webfont.eot?#iefix]
>
> will convert url
> http://accionistaseinversores.bbva.com/TLBB/fbinir/mult/stagsans-book-
> webfont.eot to complete
> URI encoding = 'ANSI_X3.4-1968'
> will convert url
> http://accionistaseinversores.bbva.com/TLBB/fbinir/mult/stagsans-book-
> webfont.eot? to local
> accionistaseinversores.bbva.com/TLBB/fbinir/mult/stagsans-book-webfont.eot
> URI encoding = 'ANSI_X3.4-1968'
>
> Tim Ruehsen
>
> Am Thursday 29 March 2012 schrieb Alejandro Supu:
> > Hi,
> >
> > I have found a bug on the latest version of the http client, wget 1.3.14
> >
> > This is how to reproduce it:
> >
> > If we save the page:
> > http://accionistaseinversores.bbva.com/TLBB/tlbb/bbvair/esp/index.jsp
> > with the following parameters: wget -k -p
> > http://accionistaseinversores.bbva.com/TLBB/tlbb/bbvair/esp/index.jsp
> >
> > On the saved "main.css" file
> > (\accionistaseinversores.bbva.com\TLBB\fbinir\css), there are files that
> > point to the remote files instead of the saved ones! For example, on line
> > 57, 68 and 79, it points to
> > http://accionistaseinversores.bbva.com/TLBB/fbinir/mult/stagsans-light-we
> > b font.eot instead of ../mult/stagsans-book-webfont.eot and this file was
> > saved to local... There are other files with the same behaviour.
> >
> > If you search the string "http" within the CSS file, you will find all
> > the pointed files to remote instead of the local SAVED ones.
> >
> > Please, tell me anything related to this bug or when it will be
> > corrected.
> >
> > THANKS!
--
