zuora_connect 1.5.303 → 1.5.304

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e9f67787b2f057558f35685a26a1c2bc192060cb
-  data.tar.gz: 6741008d64f6d109b82757b0ac4c5b860e4b6360
+  metadata.gz: 1a01603c782d3497e90daf4744c00d10470b7ac3
+  data.tar.gz: c664a8661e4e5de5fc003c321b279b3ffc51b535
 SHA512:
-  metadata.gz: f58f186f08efb88e09ab849285df46404505189018190bb5956cf8ad71d3b4914f55c66533ca94815b027bb83c9e3d53020b0feaf59409b1392ceed03707f42a
-  data.tar.gz: fdda48ded2e264ac032ea3b661f3e00485ce42046ab299429f5f565c7f3e365beb3744cf63cb91bf5c1abf54e6aef107db48aa616a5da57722d6360f74cc882f
+  metadata.gz: 90b4a6e295792b03d207649c310e95d5c45ba9799676c630894e97fa4d728f602529eff3ee94d950a87ae6222fd1a706eb827f14f94b86ba6b7b6bfe034aff3b
+  data.tar.gz: 898a5342f4040c2b1fffd766c2dddd0428dc18cb89f243e610b22b6b77d49b54b728f6466215c63df6730d5fab6394d4ed4a697f14b90362c9151d70c74bced5

data/app/models/zuora_connect/app_instance_base.rb

@@ -76,9 +76,9 @@ module ZuoraConnect
           Rails.logger.info("[#{self.id}] REFRESHING - No Time on Cookie") 
           self.refresh(session)
 
-        elsif (session["#{self.id}::last_refresh"].to_i < INSTANCE_REFRESH_WINDOW.ago.to_i) && !self.marked_for_refresh?
+        # If the cache is expired and we can aquire a refresh lock  
+        elsif (session["#{self.id}::last_refresh"].to_i < INSTANCE_REFRESH_WINDOW.ago.to_i) && self.mark_for_refresh
           Rails.logger.info("[#{self.id}] REFRESHING - Session Old by #{time_expire.abs} second")
-          self.mark_for_refresh #We will mark that we are refreshing the expired session. This will help prevent race condition between multiple requests
           self.refresh(session)
         else
           if time_expire < 0
@@ -262,9 +262,9 @@ module ZuoraConnect
         return defined?(Redis.current) ? REFRESH_TIMEOUT.to_i - Redis.current.ttl("AppInstance:#{self.id}:Refreshing") : 0
       end
 
-      def mark_for_refresh        
-        Redis.current.setex("AppInstance:#{self.id}:Refreshing", REFRESH_TIMEOUT.to_i, true) if defined?(Redis.current)
-      end
+      def mark_for_refresh
+        return defined?(Redis.current) ? Redis.current.set("AppInstance:#{self.id}:Refreshing", true, {:nx => true, :ex => REFRESH_TIMEOUT.to_i}) : true
+      end 
 
       def data_lookup(session: {})
         if defined?(PaperTrail)

data/lib/zuora_connect/version.rb

@@ -1,4 +1,4 @@
 module ZuoraConnect
-  VERSION = "1.5.303"
+  VERSION = "1.5.304"
 end
  

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: zuora_connect
 version: !ruby/object:Gem::Version
-  version: 1.5.303
+  version: 1.5.304
 platform: ruby
 authors:
 - Connect Team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-02-22 00:00:00.000000000 Z
+date: 2018-02-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: apartment