Re: [Bug-wget] [PATCH 25/25] New: --metalink-over-http Content-Type/Disposition Metalink/XML processing

[Giuseppe Scrivano]

Matthew White <address@hidden> writes:

> From ec75cdd3f0e748523c64856f01b28128f806fb16 Mon Sep 17 00:00:00 2001
> From: Matthew White <address@hidden>
> Date: Mon, 29 Aug 2016 20:59:35 +0200
> Subject: [PATCH 25/25] New: --metalink-over-http Content-Type/Disposition
>  Metalink/XML processing
>
> * src/http.c (metalink_from_http): Process the Content-Type header.
>   Add an application/metalink4+xml URL as metalink metaurl.  If the
>   option opt.content_disposition is true, the Content-Disposition's
>   filename is the metaurl's name
> * doc/wget.texi: Update --content-disposition and --metalink-over-http
> * doc/metalink-standard.txt: Update doc. Content-Type/Disposition
>   processing through --metalink-over-http. Update download naming
>   system about --trust-server-names and --content-disposition
> * testenv/Makefile.am: Add new files
> * testenv/Test-metalink-http-xml-type.py: New file. Metalink/HTTP
>   Content-Type/Disposition header automated Metalink/XML tests
> * testenv/Test-metalink-http-xml-type-trust.py: New file. Metalink/HTTP
>   Content-Type/Disposition header with --trust-server-names automated
>   Metalink/XML tests
> * testenv/Test-metalink-http-xml-type-content.py: New file. Metalink/HTTP
>   Content-Type/Disposition header with --content-disposition automated
>   Metalink/XML tests
> * testenv/Test-metalink-http-xml-type-trust-content.py: New file.
>   Metalink/HTTP Content-Type/Disposition header with --trust-server-names
>   and --content-disposition automated Metalink/XML tests
>
> Process the Content-Type header, identify an application/metalink4+xml
> file.  The Content-Disposition could provide an alternate name through
> the "filename" field for the metalink xml file.  Respectively, the cli
> options --metalink-over-http and --content-disposition are required.
>
> When Metalink/XML auto-processing, to use the Content-Disposition's
> filename, the cli option --trust-server-names is also required.
> ---
>  doc/metalink-standard.txt                          |   4 +
>  doc/wget.texi                                      |   5 +
>  src/http.c                                         |  81 ++++++++
>  testenv/Makefile.am                                |   4 +
>  testenv/Test-metalink-http-xml-type-content.py     | 222 
> +++++++++++++++++++++
>  .../Test-metalink-http-xml-type-trust-content.py   | 222 
> +++++++++++++++++++++
>  testenv/Test-metalink-http-xml-type-trust.py       | 222 
> +++++++++++++++++++++
>  testenv/Test-metalink-http-xml-type.py             | 222 
> +++++++++++++++++++++
>  8 files changed, 982 insertions(+)
>  create mode 100755 testenv/Test-metalink-http-xml-type-content.py
>  create mode 100755 testenv/Test-metalink-http-xml-type-trust-content.py
>  create mode 100755 testenv/Test-metalink-http-xml-type-trust.py
>  create mode 100755 testenv/Test-metalink-http-xml-type.py
>

I'll have a better look at this patch.

Great work on your series!  Could you also update the NEWS file with all
that is worth to say about all your new changes?

Regards,
Giuseppe