stop auto-loading Plack::Middleware::Deflater

Instead of gzipping some (mbox.gz, manifest.js.gz) responses and
leaving P::M::D to do the rest, we gzip everything ourselves,
now, so P::M::D is redundant.

diff --git a/Documentation/public-inbox-httpd.pod b/Documentation/public-inbox-httpd.pod
index c1ebfd8222b0c8a40f202de3625855b8087418b9..2f4e9e5d198bcf7fe1e51396010a50d33f683ab8 100644 (file)
--- a/Documentation/public-inbox-httpd.pod
+++ b/Documentation/public-inbox-httpd.pod
@@ -15,7 +15,6 @@ the PSGI file.
 
 If a PSGI file is not specified, L<PublicInbox::WWW> is
 loaded with a default middleware stack consisting of
-L<Plack::Middleware::Deflater>,
 L<Plack::Middleware::ReverseProxy>, and
 L<Plack::Middleware::Head>
 

diff --git a/INSTALL b/INSTALL
index 05e0f95e9142e43b5f4e31fc926c8e446b4b9cac..9f05c3f622509ec4fc8832ea13d77a575d6fb1af 100644 (file)
--- a/INSTALL
+++ b/INSTALL
@@ -100,11 +100,6 @@ Numerous optional modules are likely to be useful as well:
                                    (ensures redirects are correct when running
                                     behind nginx or Varnish)
 
-- Plack::Middleware::Deflater      deb: libplack-middleware-deflater-perl
-                                   pkg: p5 -Plack-Middleware-Deflater
-                                   rpm: perl-Plack-Middleware-Deflater
-                                   (saves bandwidth on responses)
-
 * highlight                        deb: libhighlight-perl
                                    (for syntax highlighting with coderepo)
 

diff --git a/ci/deps.perl b/ci/deps.perl
index 501f51129e7c3c69848da4ad9b43cf56d14cf0f8..77d95fc8e749071237ea03dec9ff438950393f00 100755 (executable)
--- a/ci/deps.perl
+++ b/ci/deps.perl
@@ -36,7 +36,6 @@ my $profiles = {
                Net::Server
                Plack
                Plack::Test
-               Plack::Middleware::Deflater
                Plack::Middleware::ReverseProxy
                Search::Xapian
                Socket6

diff --git a/examples/cgit.psgi b/examples/cgit.psgi
index e72e832d03be09b9c5aeedbc9620a2e1a40c4681..7ad38e2802246fe491ad5464da5964440e4894b9 100644 (file)
--- a/examples/cgit.psgi
+++ b/examples/cgit.psgi
@@ -18,14 +18,6 @@ my $pi_config = PublicInbox::Config->new;
 my $cgit = PublicInbox::Cgit->new($pi_config);
 
 builder {
-       eval {
-               enable 'Deflater',
-                       content_type => [ qw(
-                               text/html
-                               text/plain
-                               application/atom+xml
-                               )]
-       };
        eval { enable 'ReverseProxy' };
        enable 'Head';
        sub { $cgit->call($_[0]) }

diff --git a/examples/newswww.psgi b/examples/newswww.psgi
index 3cce7191da827518a6544c7db5f0d6aed2ecd9e9..52ad7043eb60db716a38ec5782dfa71a12e6226f 100644 (file)
--- a/examples/newswww.psgi
+++ b/examples/newswww.psgi
@@ -36,14 +36,6 @@ builder {
        # regular PublicInbox::WWW code:
        # see comments in examples/public-inbox.psgi for more info:
        mount '/' => builder {
-               eval {
-                       enable 'Deflater',
-                               content_type => [ qw(
-                                       text/html
-                                       text/plain
-                                       application/atom+xml
-                                       )]
-               };
                eval { enable 'ReverseProxy' };
                enable 'Head';
                sub { $www->call($_[0]) }

diff --git a/examples/public-inbox.psgi b/examples/public-inbox.psgi
index 9891a1f03e747a6a5cf92a1aec51789c40e5243b..3537be2c794b7d460948da876e43e6d68311d08e 100644 (file)
--- a/examples/public-inbox.psgi
+++ b/examples/public-inbox.psgi
@@ -22,15 +22,6 @@ my $src = $ENV{SRC_GIT_DIR}; # '/path/to/public-inbox.git'
 $src = PublicInbox::Git->new($src) if defined $src;
 
 builder {
-       eval {
-               enable 'Deflater',
-                       content_type => [ qw(
-                               text/html
-                               text/plain
-                               application/atom+xml
-                               )]
-       };
-
        # Enable to ensure redirects and Atom feed URLs are generated
        # properly when running behind a reverse proxy server which
        # sets the X-Forwarded-Proto request header.

diff --git a/examples/unsubscribe.psgi b/examples/unsubscribe.psgi
index 6a40f251d5617ed1ffee30ee58d07e84b2f656de..7b97e25322d8395aad6a3a777132e2476de64715 100644 (file)
--- a/examples/unsubscribe.psgi
+++ b/examples/unsubscribe.psgi
@@ -61,7 +61,6 @@ my $app = PublicInbox::Unsubscribe->new(
 
 builder {
        mount '/u' => builder {
-               eval { enable 'Deflater' }; # optional
                eval { enable 'ReverseProxy' }; # optional
                enable 'Head';
                sub { $app->call(@_) };

diff --git a/script/public-inbox-httpd b/script/public-inbox-httpd
index 09da505e5028952a60551f7bd2ffc25cf57c7780..b8159f3a5db233272cc10eaf83bfa8c924058e71 100755 (executable)
--- a/script/public-inbox-httpd
+++ b/script/public-inbox-httpd
@@ -27,15 +27,6 @@ my $refresh = sub {
                my $www = PublicInbox::WWW->new;
                $www->preload;
                $app = builder {
-                       eval {
-                               enable 'Deflater',
-                                       content_type => [ qw(
-                                               text/html
-                                               text/plain
-                                               application/atom+xml
-                                               )]
-                       };
-
                        eval { enable 'ReverseProxy' };
                        $@ and warn
 "Plack::Middleware::ReverseProxy missing,\n",

diff --git a/script/public-inbox.cgi b/script/public-inbox.cgi
index c766483a208cacb4dceec28e8c80795447591ed6..42ab17c9e34b8e10fb24a0410ec25a9b620a3d09 100755 (executable)
--- a/script/public-inbox.cgi
+++ b/script/public-inbox.cgi
@@ -13,14 +13,7 @@ BEGIN {
        PublicInbox::WWW->preload if $ENV{MOD_PERL};
 }
 my $www = PublicInbox::WWW->new;
-my $have_deflater = eval { require Plack::Middleware::Deflater; 1 };
 my $app = builder {
-       if ($have_deflater) {
-               enable 'Deflater',
-                       content_type => [ 'text/html', 'text/plain',
-                                       'application/atom+xml' ];
-       }
-
        # Enable to ensure redirects and Atom feed URLs are generated
        # properly when running behind a reverse proxy server which
        # sets the X-Forwarded-Proto request header.