volcano-sdk 0.9.3 → 0.9.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bb4b4a206147014ad99f42983e545238fd1c46600a127cbe932db4877afd6db9
-  data.tar.gz: 5064a928176a3dd8350d066000c405c1d03b87499b3cbc9bd0a9534b65785247
+  metadata.gz: e4fdce577eb0f97e2359db37774f29986e40c24d7d8b4bf64b67ccca4c96ea85
+  data.tar.gz: 5bb422eb1a068e899aa6de617f2bfc4bf78128698009f384c37254c753ae3ea1
 SHA512:
-  metadata.gz: 9ea3fff976b1358651b6f0117a23ccb353edb01139b9876f07b2bca473acd607550779702d0a8821c5cc612194e0fb51f279df52fd41a192b44916d60dd50844
-  data.tar.gz: 7b4761b2c1c25f131968de7662668f04c83ad90fe13532b4b7b277685fe23966ca585f578d6337bbe28dbf9c1a584dc38c0d243420f56851af559f0f6fbce085
+  metadata.gz: b0cd140b083357a515a32e3f0dccbc5280519cbaa6f08f1881f745675b979193dd6bd1ddb9e6b415ac55cdabbdcfda44a1b157e97bc2c9f0f1370292ff377922
+  data.tar.gz: 711a49849712a130ac0da5256bef2297428fd3fa2e69e91853842be89752b0e2406e66dc88c88fea78dea29bdfdfd6146701b4aa6a564638a24dad8c42904aab

data/lib/volcano/function_resolution.rb

@@ -14,7 +14,14 @@ module Volcano
     Resolution = Data.define(:function_id, :invoke_url)
 
     # A cached resolve result. A nil resolution is a remembered miss.
-    Outcome = Data.define(:resolution)
+    Outcome = Data.define(:resolution, :failure)
+
+    Failure = Data.define(:message, :code, :retry_after) do
+      def exception
+        Error::NotFoundError.new(message.dup, status: 404, code: code&.dup, retry_after: retry_after)
+      end
+    end
+    private_constant :Failure
 
     Entry = Data.define(:outcome, :expires_at)
     private_constant :Entry
@@ -72,13 +79,15 @@ module Volcano
 
       # Caches a resolution for the server-advertised lifetime.
       def store(api_url, authorization, name, resolution, ttl_seconds)
-        write([api_url, authorization, name], Outcome.new(resolution: resolution), ttl_seconds)
+        write([api_url, authorization, name], Outcome.new(resolution: resolution, failure: nil), ttl_seconds)
       end
 
       # Remembers briefly that a name does not resolve, so a caller retrying an
       # unknown name in a loop does not re-ask the server on every attempt.
-      def store_missing(api_url, authorization, name)
-        write([api_url, authorization, name], Outcome.new(resolution: nil), NEGATIVE_TTL_SECONDS)
+      def store_missing(api_url, authorization, name, error)
+        failure = Failure.new(message: error.message.dup.freeze, code: error.code&.dup&.freeze,
+                              retry_after: error.retry_after)
+        write([api_url, authorization, name], Outcome.new(resolution: nil, failure: failure), NEGATIVE_TTL_SECONDS)
       end
 
       # Drops one cached resolution that turned out to be stale.

data/lib/volcano/functions.rb

@@ -88,15 +88,17 @@ module Volcano
       resolved = Transport.invoke do
         @transport.resolve_function_for_invocation(authorization: authorization, name: name)
       end
-      FunctionResolution.store_missing(@api_url, authorization, name) if resolved.status == 404
       payload = Transport.body(resolved, 200)
       resolution = resolved_resolution(payload)
       FunctionResolution.store(@api_url, authorization, name, resolution, cache_ttl(payload))
       resolution
+    rescue Error::NotFoundError => e
+      FunctionResolution.store_missing(@api_url, authorization, name, e)
+      raise
     end
 
     def cached_resolution(cached)
-      cached.resolution || raise(Error::NotFoundError.new('Function was not found', status: 404))
+      cached.resolution || raise(cached.failure.exception)
     end
 
     def validate_invocation(name, payload)

data/lib/volcano/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Volcano
-  VERSION = '0.9.3'
+  VERSION = '0.9.4'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: volcano-sdk
 version: !ruby/object:Gem::Version
-  version: 0.9.3
+  version: 0.9.4
 platform: ruby
 authors:
 - Kong