Cleaner to set Cache-Control: private

metacpan · Feb 22, 2017 · 1be21de · 1be21de
1 parent 3029017
commit 1be21de
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/lib/MetaCPAN/Web/Controller/Account.pm b/lib/MetaCPAN/Web/Controller/Account.pm
@@ -9,10 +9,6 @@ BEGIN { extends 'MetaCPAN::Web::Controller' }
 sub auto : Private {
     my ( $self, $c ) = @_;
 
-    # Keep everything here as specific to the user
-    # this is for proxies
-    $c->res->header( Vary => 'Cookie' );
-
     $c->cdn_never_cache(1);
 
     if ( my $token = $c->token ) {

diff --git a/t/controller/account.t b/t/controller/account.t
@@ -36,8 +36,9 @@ test_psgi app, sub {
             'GET /account/profile without token'
         );
         is( $res->code, 403, '... and the user cannot get in' );
-        is( $res->header('Vary'),
-            'Cookie', '... and the Vary header for proxies is there' );
+        is( $res->header('Cache-Control'),
+            'private',
+            '... and the private Cache-Control header for proxies is there' );
         is( $authenticate_args, undef,
             '... and we did not try to authenticate' );