Fix refresh of Bedrock models and remove some expired credentials from cassettes (#89)

Author: tpaulshippy

lib/ruby_llm/providers/bedrock.rb

@@ -25,17 +25,17 @@ def api_base
       end
 
       def post(url, payload)
-        signature = sign_request("#{connection.url_prefix}#{url}", payload)
+        signature = sign_request("#{connection.url_prefix}#{url}", payload:)
         connection.post url, payload do |req|
           req.headers.merge! build_headers(signature.headers, streaming: block_given?)
 
           yield req if block_given?
         end
       end
 
-      def sign_request(url, payload)
+      def sign_request(url, method: :post, payload: nil)
         signer = create_signer
-        request = build_request(url, payload)
+        request = build_request(url, method:, payload:)
         signer.sign_request(request)
       end
 
@@ -49,12 +49,12 @@ def create_signer
                             })
       end
 
-      def build_request(url, payload)
+      def build_request(url, method: :post, payload: nil)
         {
           connection: connection,
-          http_method: :post,
+          http_method: method,
           url: url || completion_url,
-          body: JSON.generate(payload, ascii_only: false) || ''
+          body: payload ? JSON.generate(payload, ascii_only: false) : nil
         }
       end
 

lib/ruby_llm/providers/bedrock/models.rb

@@ -8,7 +8,8 @@ module Models
         def list_models
           @connection = nil # reset connection since base url is different
           @api_base = "https://bedrock.#{RubyLLM.config.bedrock_region}.amazonaws.com"
-          signature = sign_request(models_url, nil)
+          full_models_url = "#{@api_base}/#{models_url}"
+          signature = sign_request(full_models_url, method: :get)
           response = connection.get(models_url) do |req|
             req.headers.merge! signature.headers
           end