[anaconda][master][PATCH] Replace urlgrabber usage by python-requests
Brian C. Lane
bcl at redhat.com
Thu Jan 22 19:08:01 UTC 2015
On Thu, Jan 22, 2015 at 07:38:54PM +0100, Martin Kolman wrote:
> Replace the Python 2-only urlgrabber with the Python 3 compatible (and much nicer ;-) )
> python-requests library.
>
> The requests API maps quite nicely to the stuff we did with urlgrabber and should
> have the same end result & behavior as before.
>
> The only potential mismatch I've found I found is in SSL certificate verification,
> the urlgrabber API has two options - ssl_verify_host and ssl_verify_peer, while
> request just have a single verify option. We used to set both options always to True
> or to False with urlgrabber, so unless requests are skipping some verification urlgrabber
> did, so this should be probably fine.
>
> Certificate validation itself seem to work fine as far as I can tell - connections to
> server with valid certificates work fine and servers with self-signed certificates raise
> an exception.
>
> Martin Kolman (1):
> Replace python-urlgrabber with python-requests (#1141242)
>
> anaconda.spec.in | 5 ++-
> pyanaconda/kickstart.py | 13 +++++---
> pyanaconda/packaging/__init__.py | 36 ++++++++++-----------
> pyanaconda/packaging/livepayload.py | 64 ++++++++++++++++++++-----------------
> 4 files changed, 63 insertions(+), 55 deletions(-)
>
> --
> 2.1.0
Looks good. Have you tried it with really large (like 8G+) images yet?
urlgrabber had the habit of getting slower the more data it read.
--
Brian C. Lane | Anaconda Team | IRC: bcl #anaconda | Port Orchard, WA (PST8PDT)
More information about the anaconda-patches
mailing list