Compress the appstore requests by default

In test it reduced the transfered data from 5 MB to 2 MB. This should reduce the load on the appstore significantly.

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

diff --git a/lib/private/App/AppStore/Fetcher/Fetcher.php b/lib/private/App/AppStore/Fetcher/Fetcher.php
index 0691bf23585d9434c087351ae7a6363f7d726424..2c7b7548bef5299135926d2f8227777a8052bb82 100644 (file)
--- a/lib/private/App/AppStore/Fetcher/Fetcher.php
+++ b/lib/private/App/AppStore/Fetcher/Fetcher.php
@@ -96,12 +96,11 @@ abstract class Fetcher {
 
                $options = [
                        'timeout' => 10,
+                       'headers' => ['Accept-Encoding' => 'gzip'],
                ];
 
                if ($ETag !== '') {
-                       $options['headers'] = [
-                               'If-None-Match' => $ETag,
-                       ];
+                       $options['headers']['If-None-Match'] = $ETag;
                }
 
                $client = $this->clientService->newClient();

diff --git a/tests/lib/App/AppStore/Fetcher/FetcherBase.php b/tests/lib/App/AppStore/Fetcher/FetcherBase.php
index 57124c3712d699e361ed35cb47ce83e5f2082d7f..aa54bbb5c268d76c3e10403bf19fd0b885a68ddc 100644 (file)
--- a/tests/lib/App/AppStore/Fetcher/FetcherBase.php
+++ b/tests/lib/App/AppStore/Fetcher/FetcherBase.php
@@ -249,7 +249,15 @@ abstract class FetcherBase extends TestCase {
                $client
                        ->expects($this->once())
                        ->method('get')
-                       ->with($this->endpoint)
+                       ->with(
+                               $this->equalTo($this->endpoint),
+                               $this->equalTo([
+                                       'timeout' => 10,
+                                       'headers' => [
+                                               'Accept-Encoding' => 'gzip',
+                                       ]
+                               ])
+                       )
                        ->willReturn($response);
                $response
                        ->expects($this->once())
@@ -342,7 +350,15 @@ abstract class FetcherBase extends TestCase {
                $client
                        ->expects($this->once())
                        ->method('get')
-                       ->with($this->endpoint)
+                       ->with(
+                               $this->equalTo($this->endpoint),
+                               $this->equalTo([
+                                       'timeout' => 10,
+                                       'headers' => [
+                                               'Accept-Encoding' => 'gzip',
+                                       ]
+                               ])
+                       )
                        ->willReturn($response);
                $response
                        ->expects($this->once())
@@ -430,7 +446,15 @@ abstract class FetcherBase extends TestCase {
                $client
                        ->expects($this->once())
                        ->method('get')
-                       ->with($this->endpoint)
+                       ->with(
+                               $this->equalTo($this->endpoint),
+                               $this->equalTo([
+                                       'timeout' => 10,
+                                       'headers' => [
+                                               'Accept-Encoding' => 'gzip',
+                                       ]
+                               ])
+                       )
                        ->willReturn($response);
                $response
                        ->expects($this->once())
@@ -495,7 +519,15 @@ abstract class FetcherBase extends TestCase {
                $client
                        ->expects($this->once())
                        ->method('get')
-                       ->with($this->endpoint)
+                       ->with(
+                               $this->equalTo($this->endpoint),
+                               $this->equalTo([
+                                       'timeout' => 10,
+                                       'headers' => [
+                                               'Accept-Encoding' => 'gzip',
+                                       ]
+                               ])
+                       )
                        ->willThrowException(new \Exception());
 
                $this->assertSame([], $this->fetcher->get());
@@ -552,7 +584,8 @@ abstract class FetcherBase extends TestCase {
                                $this->equalTo([
                                        'timeout' => 10,
                                        'headers' => [
-                                               'If-None-Match' => '"myETag"'
+                                               'Accept-Encoding' => 'gzip',
+                                               'If-None-Match' => '"myETag"',
                                        ]
                                ])
                        )->willReturn($response);
@@ -624,6 +657,7 @@ abstract class FetcherBase extends TestCase {
                                $this->equalTo([
                                        'timeout' => 10,
                                        'headers' => [
+                                               'Accept-Encoding' => 'gzip',
                                                'If-None-Match' => '"myETag"',
                                        ]
                                ])
@@ -710,6 +744,9 @@ abstract class FetcherBase extends TestCase {
                                $this->equalTo($this->endpoint),
                                $this->equalTo([
                                        'timeout' => 10,
+                                       'headers' => [
+                                               'Accept-Encoding' => 'gzip',
+                                       ],
                                ])
                        )
                        ->willReturn($response);