Fix user delegation keys not refreshing

Fix #14

Author: JoeDupuis

CHANGELOG.md

@@ -1,5 +1,9 @@
 ## [Unreleased]
 
+## [0.5.6] 2025-01-17
+
+- Fix user delegation key not refreshing (#14)
+
 ## [0.5.5] 2025-01-10
 
 - Allow lazy loading the signer

Gemfile.lock

@@ -97,7 +97,7 @@ GIT
 PATH
   remote: .
   specs:
-    azure-blob (0.5.5)
+    azure-blob (0.5.6)
       rexml
 
 GEM

lib/azure_blob/identity_token.rb

@@ -19,7 +19,7 @@ def initialize(principal_id: nil)
     end
 
     def to_s
-      refresh if expired?
+      refresh
       token
     end
 
@@ -30,6 +30,7 @@ def expired?
     end
 
     def refresh
+      return unless expired?
       headers =  { "Metadata" => "true" }
       headers["X-IDENTITY-HEADER"] = ENV["IDENTITY_HEADER"] if ENV["IDENTITY_HEADER"]
 

lib/azure_blob/user_delegation_key.rb

@@ -15,6 +15,7 @@ def initialize(account_name:, signer:)
     end
 
     def to_s
+      refresh
       user_delegation_key
     end
 

lib/azure_blob/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module AzureBlob
-  VERSION = "0.5.5"
+  VERSION = "0.5.6"
 end

test/client/test_user_delegation_key.rb

@@ -42,4 +42,19 @@ def test_refresh_when_over_expiration_buffer
 
     refute_equal initial_expiry, delegation_key.signed_expiry
   end
+
+  def test_to_s_refresh_automatically
+    now = Time.now
+    after_expiration_buffer = now + 21601
+    Time.stub :now,  now do
+      @delegation_key = AzureBlob::UserDelegationKey.new(account_name:, signer:)
+    end
+
+    initial_expiry = delegation_key.signed_expiry
+    Time.stub :now,  after_expiration_buffer do
+      delegation_key.to_s
+    end
+
+    refute_equal initial_expiry, delegation_key.signed_expiry
+  end
 end