update-webkit-auxiliary-libs can't download WebKitAuxiliaryLibrary.zip due to 403...
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 19 Oct 2017 00:46:15 +0000 (00:46 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 19 Oct 2017 00:46:15 +0000 (00:46 +0000)
https://bugs.webkit.org/show_bug.cgi?id=178381

Patch by Fujii Hironori <Hironori.Fujii@sony.com> on 2017-10-18
Reviewed by Alexey Proskuryakov.

The web server developer.apple.com seems to reject HTTP requests
without Accept header field.

* Scripts/update-webkit-dependency: Add 'Accept' header field to requests.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@223643 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/Scripts/update-webkit-dependency

index 1a51fc2..f0f3393 100644 (file)
@@ -1,3 +1,15 @@
+2017-10-18  Fujii Hironori  <Hironori.Fujii@sony.com>
+
+        update-webkit-auxiliary-libs can't download WebKitAuxiliaryLibrary.zip due to 403 Forbidden
+        https://bugs.webkit.org/show_bug.cgi?id=178381
+
+        Reviewed by Alexey Proskuryakov.
+
+        The web server developer.apple.com seems to reject HTTP requests
+        without Accept header field.
+
+        * Scripts/update-webkit-dependency: Add 'Accept' header field to requests.
+
 2017-10-18  Aakash Jain  <aakash_jain@apple.com>
 
         webkitpy tests should have --json-output option
index 1b04d77..e366daf 100755 (executable)
@@ -95,7 +95,7 @@ print "Checking for newer version of $zipFile...\n";
 # errors for those requests. Instead we can just set a size limit for a GET request (which we can also set the range for
 # to request just the first byte) so that we don't have to retrieve the entire file.
 $ua->max_size(0);
-my $response = $ua->get($libsURL, "Range", "bytes=0-0");
+my $response = $ua->get($libsURL, Accept => '*/*', Range => "bytes=0-0");
 $ua->max_size(undef);
 
 if (! $response->is_success) {
@@ -191,8 +191,10 @@ if (defined $lastModifiedTime && defined $oldLastModifiedTime
 
 print "Downloading $zipFile...\n\n";
 print "$libsURL\n";
-my $result = getstore($libsURL, File::Spec->catfile($tmpAbsDir, $zipFile));
-die "Couldn't download $zipFile!" if is_error($result);
+my $request = HTTP::Request->new(GET => $libsURL);
+$request->header(Accept => "*/*");
+$response = $ua->request($request, File::Spec->catfile($tmpAbsDir, $zipFile));
+die "Couldn't download $zipFile!" if is_error($response->code);
 
 my $zip = Archive::Zip->new(File::Spec->catfile($tmpAbsDir, $zipFile));
 
@@ -211,7 +213,7 @@ if (! $zip->memberNamed($prefixDirectoryPathInZipFile)) {
     }
 }
 
-$result = $zip->extractTree("", $tmpAbsDir);
+my $result = $zip->extractTree("", $tmpAbsDir);
 die "Couldn't unzip $zipFile." if $result != AZ_OK;
 
 print "\nInstalling $file...\n";