alephant-broker 3.4.1 → 3.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 66e7169ca6dccf162a18d99cd370e664743f7368
-  data.tar.gz: ed5178f9dc74b2f4b339ceffe9c2a1aff969c566
+  metadata.gz: fda6d6a6605e71b9eb66abf08dddb9d5e1baab44
+  data.tar.gz: cec4b99897a6bcbf1265586d48ef6bac5cd182e0
 SHA512:
-  metadata.gz: d761eaba217f4f256ccd1544063d237624af19f4609d9880cfc09694d15c8767c80b423bc8974e6d235d9603c28e808f79d9f358a61b59bad4ca93e0674f324b
-  data.tar.gz: 12b02ddece7cdd609f018b99a080e735bc57993ec472409bc492f6a68418f0bbd95c8635e37154ec92b87c2d62d56f33c999fcce06045df17b0a32cb6ea7df49
+  metadata.gz: de0065a472ec3619e4ff4ef17b05d797bf80d3a9653b63d635a37a9c76c2a5f86b01ae901d628cc8280d9c4b1437c2f97b0e48522816bd26b61147b805216edf
+  data.tar.gz: af56c9641a638cb8c2683f713774701530f7972e6bf53e29efe9d46ba3afab8832f353d9899128f1de7965160e061c9f0e58c982792aee35a1cb88f3a8c23201

data/lib/alephant/broker/response/base.rb

@@ -1,7 +1,8 @@
-require 'alephant/broker/errors/invalid_cache_key'
-require 'alephant/logger'
-require 'aws-sdk'
-require 'ostruct'
+require "alephant/broker/errors/invalid_cache_key"
+require "alephant/logger"
+require "aws-sdk"
+require "ostruct"
+require "date"
 
 module Alephant
   module Broker
@@ -12,9 +13,9 @@ module Alephant
         attr_reader :content, :headers, :status
 
         STATUS_CODE_MAPPING = {
-          200 => 'ok',
-          404 => 'Not found',
-          500 => 'Error retrieving content'
+          200 => "ok",
+          404 => "Not found",
+          500 => "Error retrieving content"
         }
 
         def initialize(status = 200, content_type = "text/html")
@@ -22,8 +23,7 @@ module Alephant
           @headers = { "Content-Type" => content_type }
           @status  = status
 
-          log if status !~ /200/
-
+          log_status
           setup
         end
 
@@ -33,11 +33,27 @@ module Alephant
 
         private
 
+        def log_status
+          add_no_cache_headers if status !~ /200/
+        end
+
+        def add_no_cache_headers
+          headers.merge!(
+            "Cache-Control" => "no-cache, must-revalidate",
+            "Pragma"        => "no-cache",
+            "Expires"       => Date.today.prev_year.httpdate
+          )
+          log
+        end
+
         def log
-          logger.metric(:name => "BrokerResponse#{status}", :unit => "Count", :value => 1)
+          logger.metric(
+            :name  => "BrokerResponse#{status}",
+            :unit  => "Count",
+            :value => 1
+          )
         end
       end
     end
   end
 end
-

data/lib/alephant/broker/version.rb

@@ -1,5 +1,5 @@
 module Alephant
   module Broker
-    VERSION = "3.4.1"
+    VERSION = "3.5.0"
   end
 end

data/spec/integration/rack_spec.rb

@@ -59,6 +59,9 @@ describe Alephant::Broker::Application do
     before { get "/banana" }
     specify { expect(last_response.status).to eql 404 }
     specify { expect(last_response.body).to eq "Not found" }
+    specify { expect(last_response.headers).to include("Cache-Control") }
+    specify { expect(last_response.headers).to include("Pragma") }
+    specify { expect(last_response.headers).to include("Expires") }
   end
 
   describe "Component endpoint '/component/...'" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: alephant-broker
 version: !ruby/object:Gem::Version
-  version: 3.4.1
+  version: 3.5.0
 platform: ruby
 authors:
 - Steven Jack