volcano-sdk 0.7.0 → 0.7.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a36dee9d50b8138335752de5e3fdb5007ecd5dfa66547150190de691a048052d
-  data.tar.gz: 97edb400526abafd980ec19278304f80cff3ead5010d7704da7b55128390879e
+  metadata.gz: 9d85e7ec311f812ca5267cec7b463ebd65ff82ba44f4ce06e71f23085f7932ea
+  data.tar.gz: 772266f8fe5398d085442a03bcf2544468016fc335eb61fb0c1ba04ba47dde35
 SHA512:
-  metadata.gz: 5c3e7ed134b563780e8c1d64ba26f30f7ae9563d7c2ac219a265dabbf6aa04544600ef1a963abd09f9aa3f4efc2a6e32da0adeaf9a2a45905bd793daffd4101d
-  data.tar.gz: c2abfe5361e2ccff60f6f76a77aa4e2eedc0a033d1ef7649a63c5a6fa8f1edf0996c8141cc5eede6c08e5848e24fa8159e5a550b12edc5a32d16ea6dbf4311aa
+  metadata.gz: 2718f112a8e2264f826aa4f3bc89c61f4492ee6034c0913d0e8b64e3d5662e8b89579af0dfbe11a88b2c92fee01c8f7508f90e347a0a7b14f7e4aa3fd40433f4
+  data.tar.gz: 773a396b12e6c1b02ed03c5c3d8b4356b39bdac11a222bfecd2817ff0fe6ceee7644d367bb4255f6da90652cf45782efc4fc922e5d2501042ba07d1f1a2847b7

data/README.md

@@ -487,7 +487,8 @@ syntax. Both methods require an active user session.
 rows = client.database("main").from("items").select("*").eq("slug", "a").execute
 ```
 
-Database selects, inserts, updates, deletes, and log reads refresh the captured
+Database selects, inserts, updates, deletes, log reads, and authenticated storage
+requests (including upload sessions and parts) refresh the captured
 session after an HTTP 401 and retry the same request once. Concurrent requests reuse a successful
 refresh for that session.
 Replacing or signing out the session before replay, or while replay is in flight,

data/lib/volcano/auth_refresh.rb

@@ -10,10 +10,10 @@ module Volcano
       refresh_captured_session(binding)
     end
 
-    def session_request
-      binding = @client.capture_session_binding
+    def session_request(binding: @client.capture_session_binding)
       raise Error::AuthenticationError, 'No active session' unless binding.last
 
+      binding = owned_session_binding(binding)
       response = yield(binding.last.access_token)
       return response unless response.status == 401
 
@@ -65,10 +65,10 @@ module Volcano
       active = @client.capture_session_binding
       raise Error::AuthenticationError, 'No active session' if rejected_refresh?(binding, active)
 
-      generation, active_lineage, current = active
+      _, active_lineage, current = active
       raise Error::SessionChangedError unless current && active_lineage == binding[1]
 
-      [generation, current]
+      active
     end
 
     def rejected_refresh?(binding, active)

data/lib/volcano/client.rb

@@ -49,8 +49,8 @@ module Volcano
       session.access_token
     end
 
-    def session_request(&)
-      @auth.session_request(&)
+    def session_request(...)
+      @auth.session_request(...)
     end
 
     alias session_read session_request

data/lib/volcano/storage.rb

@@ -28,7 +28,7 @@ module Volcano
     def initialize(client, transport, name, api_url:, anon_key:)
       @client = client
       @transport = transport
-      @name = name
+      @name = name.dup.freeze
       @api_url = api_url.dup.freeze
       @anon_key = anon_key.dup.freeze
       freeze
@@ -36,12 +36,16 @@ module Volcano
 
     def upload(path, value, content_type: nil)
       mime_type = upload_content_type(content_type)
-      response = Transport.invoke do
+      path = storage_paths(path).first
+      binding = @client.capture_session_binding
+      @client.session_token
+      content = upload_bytes(value).freeze
+      response = storage_request(binding: binding) do |token|
         @transport.upload_storage_object(
-          authorization: @client.session_token,
+          authorization: token,
           bucket_name: @name,
           path: path,
-          data: upload_bytes(value),
+          data: content,
           content_type: mime_type
         )
       end
@@ -49,9 +53,11 @@ module Volcano
     end
 
     def download(path, range: nil)
-      response = Transport.invoke do
+      path = storage_paths(path).first
+      range = range&.dup&.freeze
+      response = storage_request do |token|
         @transport.download_storage_object(
-          authorization: @client.session_token,
+          authorization: token,
           bucket_name: @name,
           path: path,
           byte_range: range
@@ -63,9 +69,11 @@ module Volcano
     end
 
     def list(prefix = '', limit: nil, cursor: nil)
-      response = Transport.invoke do
+      prefix = prefix.dup.freeze
+      cursor = cursor&.dup&.freeze
+      response = storage_request do |token|
         @transport.list_storage_objects(
-          authorization: @client.session_token,
+          authorization: token,
           bucket_name: @name,
           prefix: prefix,
           limit: limit,
@@ -82,6 +90,10 @@ module Volcano
 
     private
 
+    def storage_request(binding: @client.capture_session_binding)
+      @client.session_request(binding: binding) { |token| Transport.invoke { yield(token) } }
+    end
+
     def upload_content_type(value)
       return if value.nil?
       return value.dup.freeze if value.is_a?(String) && value.match?(/\A[\x20-\x7e]+\z/) && !value.strip.empty?

data/lib/volcano/storage_mutations.rb

@@ -5,16 +5,16 @@ module Volcano
   class StorageBucket
     def remove(paths)
       deleted = storage_paths(paths)
-      authorization = @client.session_token
-      deleted.each { |path| delete_path(path, authorization) }
+      binding = @client.capture_session_binding
+      deleted.each { |path| delete_path(path, binding) }
       deleted
     end
 
     def move(from_path, to_path)
       source, destination = storage_paths([from_path, to_path])
-      response = Transport.invoke do
+      response = storage_request do |token|
         @transport.move_storage_object(
-          authorization: @client.session_token,
+          authorization: token,
           bucket_name: @name,
           from_path: source,
           to_path: destination
@@ -25,9 +25,9 @@ module Volcano
 
     def copy(from_path, to_path)
       source, destination = storage_paths([from_path, to_path])
-      response = Transport.invoke do
+      response = storage_request do |token|
         @transport.copy_storage_object(
-          authorization: @client.session_token,
+          authorization: token,
           bucket_name: @name,
           from_path: source,
           to_path: destination
@@ -39,9 +39,9 @@ module Volcano
     def update_visibility(path, public:)
       object_path = storage_paths(path).fetch(0)
       visibility = visibility_value(public)
-      response = Transport.invoke do
+      response = storage_request do |token|
         @transport.update_storage_object_visibility(
-          authorization: @client.session_token,
+          authorization: token,
           bucket_name: @name,
           path: object_path,
           is_public: visibility
@@ -61,10 +61,10 @@ module Volcano
       path_list.map { |path| path.dup.freeze }.freeze
     end
 
-    def delete_path(path, authorization)
-      response = Transport.invoke do
+    def delete_path(path, binding)
+      response = storage_request(binding: binding) do |token|
         @transport.delete_storage_object(
-          authorization: authorization,
+          authorization: token,
           bucket_name: @name,
           path: path
         )

data/lib/volcano/storage_upload_sessions.rb

@@ -15,9 +15,9 @@ module Volcano
         total_size: total_size,
         part_size: part_size
       )
-      response = Transport.invoke do
+      response = storage_request do |token|
         @transport.create_upload_session(
-          authorization: @client.session_token,
+          authorization: token,
           bucket_name: @name,
           request: request
         )
@@ -32,9 +32,9 @@ module Volcano
         part_number: part_number,
         data: upload_bytes(data)
       )
-      response = Transport.invoke do
+      response = storage_request do |token|
         @transport.upload_part(
-          authorization: @client.session_token,
+          authorization: token,
           bucket_name: @name,
           request: request
         )
@@ -44,9 +44,9 @@ module Volcano
 
     def complete_upload_session(path, session_id:)
       request = UploadSessionReference.new(path: path, session_id: session_id)
-      response = Transport.invoke do
+      response = storage_request do |token|
         @transport.complete_upload_session(
-          authorization: @client.session_token,
+          authorization: token,
           bucket_name: @name,
           request: request
         )
@@ -56,9 +56,9 @@ module Volcano
 
     def get_upload_session(path, session_id:)
       request = UploadSessionReference.new(path: path, session_id: session_id)
-      response = Transport.invoke do
+      response = storage_request do |token|
         @transport.get_upload_session(
-          authorization: @client.session_token,
+          authorization: token,
           bucket_name: @name,
           request: request
         )
@@ -68,9 +68,9 @@ module Volcano
 
     def abort_upload_session(path, session_id:)
       request = UploadSessionReference.new(path: path, session_id: session_id)
-      response = Transport.invoke do
+      response = storage_request do |token|
         @transport.abort_upload_session(
-          authorization: @client.session_token,
+          authorization: token,
           bucket_name: @name,
           request: request
         )

data/lib/volcano/version.rb

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

metadata

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