about summary refs log tree commit homepage
path: root/examples
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2016-02-28 00:57:11 +0000
committerEric Wong <e@80x24.org>2016-02-28 05:14:59 +0000
commit1e25da9df6f7721f99c3d4beb8444ce1165c8097 (patch)
treecc39bd8ffb1481e3525f366fb86af465645c49cb /examples
parente5dd8584b5e798b3c391e4f3e71a8c9e97c0f9d9 (diff)
downloadpublic-inbox-1e25da9df6f7721f99c3d4beb8444ce1165c8097.tar.gz
Not everybody will be running this behind a ReverseProxy;
but it's probably the likely configuration.  Anyways,
warn about this and also about Deflater being missing.
Diffstat (limited to 'examples')
-rw-r--r--examples/public-inbox.psgi20
1 files changed, 13 insertions, 7 deletions
diff --git a/examples/public-inbox.psgi b/examples/public-inbox.psgi
index acceba3d..0d11c19f 100644
--- a/examples/public-inbox.psgi
+++ b/examples/public-inbox.psgi
@@ -9,21 +9,27 @@ use PublicInbox::WWW;
 PublicInbox::WWW->preload;
 use Plack::Request;
 use Plack::Builder;
-my $have_deflater = eval { require Plack::Middleware::Deflater; 1 };
 my $www = PublicInbox::WWW->new;
 builder {
         enable 'Chunked';
-        if ($have_deflater) {
+        eval {
                 enable 'Deflater',
-                        content_type => [ 'text/html', 'text/plain',
-                                        'application/atom+xml' ];
-        }
-
+                        content_type => [ qw(
+                                text/html
+                                text/plain
+                                application/atom+xml
+                                )]
+        };
+        $@ and warn
+"Plack::Middleware::Deflater missing, bandwidth will be wasted\n";
         # Enable to ensure redirects and Atom feed URLs are generated
         # properly when running behind a reverse proxy server which
         # sets X-Forwarded-For and X-Forwarded-Proto request headers.
         # See Plack::Middleware::ReverseProxy documentation for details
-        enable 'ReverseProxy';
+        eval { enable 'ReverseProxy' };
+        $@ and warn
+"Plack::Middleware::ReverseProxy missing,\n",
+"URL generation for redirects may be wrong if behind a reverse proxy\n";
 
         enable 'Head';
         sub { $www->call(@_) };