[gnome-continuous-yocto/gnomeostree-3.28-rocko: 7058/8267] bitbake: wget: fix FusionForge workaround



commit 780e9fb87796bff5445179412f69e80b9438f323
Author: Ross Burton <ross burton intel com>
Date:   Sat Jul 29 18:23:33 2017 +0100

    bitbake: wget: fix FusionForge workaround
    
    My previous assertion about FusionForge appears to have been wrong, or
    FusionForge has changed behaviour, or both.
    
    FusionForge now mandates that downloads have the Accept header set, despite that
    header being optional, and returns a 406 Not Acceptable error if it isn't set.
    As we were pretending that 406 was actually 405 (Moved) and tried to handle it as a
    redirect this results in an infinite loop until Python kills the recursion.
    
    Delete the handling of 406 as 405, and pass Accept: */* in the headers.
    
    (Bitbake rev: bb70ae0c9aac5ec688026d23a64ac0cac1947187)
    
    Signed-off-by: Ross Burton <ross burton intel com>
    Signed-off-by: Richard Purdie <richard purdie linuxfoundation org>

 bitbake/lib/bb/fetch2/wget.py |    9 +++------
 1 files changed, 3 insertions(+), 6 deletions(-)
---
diff --git a/bitbake/lib/bb/fetch2/wget.py b/bitbake/lib/bb/fetch2/wget.py
index 8ee9769..7c49c2b 100644
--- a/bitbake/lib/bb/fetch2/wget.py
+++ b/bitbake/lib/bb/fetch2/wget.py
@@ -283,11 +283,6 @@ class Wget(FetchMethod):
             """
             http_error_403 = http_error_405
 
-            """
-            Some servers (e.g. FusionForge) returns 406 Not Acceptable when they
-            actually mean 405 Method Not Allowed.
-            """
-            http_error_406 = http_error_405
 
         class FixedHTTPRedirectHandler(urllib.request.HTTPRedirectHandler):
             """
@@ -316,7 +311,9 @@ class Wget(FetchMethod):
             uri = ud.url.split(";")[0]
             r = urllib.request.Request(uri)
             r.get_method = lambda: "HEAD"
-
+            # Some servers (FusionForge, as used on Alioth) require that the
+            # optional Accept header is set.
+            r.add_header("Accept", "*/*")
             def add_basic_auth(login_str, request):
                 '''Adds Basic auth to http request, pass in login:password as string'''
                 import base64


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]