foobara-http-command-connector 0.0.18 → 0.0.20

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bf60ac4b2586bf6a354520e1f5628d5d15553c8d1b77c5928ccf9c2dc19d3ba8
-  data.tar.gz: a86337c5edea3aff592f62cb756afac76e84fd7cce39b41f8b6bfc0b74a728e6
+  metadata.gz: 84f98d62ccf5d4607dfa7fcec72c146783afdcaf05fea4c6a783886d51a76446
+  data.tar.gz: d5d43e938ff41147061148b8fdac2bc15f54dfa8303f3b3225079d49d880d200
 SHA512:
-  metadata.gz: 7ba72c2c08688e6b08aeb76f9fcf4fe0c28f4c727093d051383f907999cedd64c773e70329079698cbbd4cbfdc390d9a27c1c73db8fd20ff2a4ba18cb252fbd3
-  data.tar.gz: fd2142b3f17b81148a9db464d761bb0a97368fdaaf421fd3392063e84aa4b919b6dbbd2a66089abbc1a66f0de444de5f65f521db67b50d2edee9c715e78924e8
+  metadata.gz: 40d905a7991b607b9b1dae7604969cb1eb40f357c1850bc743f2200a27badce71d6fc76d13680c348216047ec3c67587e83f23730657319839301c5c536e82bc
+  data.tar.gz: 3a81ef2395e42ad2027b86d5e21651b3e9ba95db4be779964d0a954e025336a67bb01ac0aa91ab539d58076dcbd2663bdc5196cf10c177849f252bffed462f81

data/CHANGELOG.md

@@ -1,3 +1,12 @@
+## [0.0.20] - 2025-04-14
+
+- Handle more cors scenarios
+
+## [0.0.19] - 2025-03-31
+
+- Add SetHeader response mutator
+- Use / instead of :: for paths
+
 ## [0.0.18] - 2025-03-29
 
 - Implement SetInputsFromHeader request mutator and MoveAttributeToHeader response mutator

data/src/http/commands/describe.rb

@@ -14,7 +14,7 @@ module Foobara
           end
 
           def in_detached_context(&)
-            Thread.foobara_with_var("foobara_manifest_context", detached: true, &)
+            TypeDeclarations.with_manifest_context(detached: true, &)
           end
 
           def build_manifest

data/src/http/commands/get_options.rb

@@ -4,11 +4,49 @@ module Foobara
       module Commands
         # TODO: this is a bit of a hack, just a total no-op... shouldn't really need this command at all ideally
         class GetOptions < Foobara::Command
+          inputs do
+            request :duck, :required
+          end
           result :string
 
           def execute
+            initialize_response_headers
+
+            set_allow_methods
+            set_allow_headers
+            set_access_control_max_age
+
+            # TODO: what's with this empty string?
             ""
           end
+
+          def initialize_response_headers
+            request.response_headers ||= {}
+          end
+
+          def set_allow_methods
+            allow_methods = ENV.fetch("FOOBARA_HTTP_RESPONSE_HEADER_ACCESS_CONTROL_ALLOW_METHODS", nil)
+            if allow_methods
+              request.response_headers["access-control-allow-methods"] = allow_methods
+            end
+          end
+
+          def set_allow_headers
+            allow_headers = ENV.fetch("FOOBARA_HTTP_RESPONSE_HEADER_ACCESS_CONTROL_ALLOW_HEADERS", nil)
+            if allow_headers
+              if allow_headers == "*"
+                allow_headers = request.headers["HTTP_ACCESS_CONTROL_REQUEST_HEADERS"]
+              end
+              request.response_headers["access-control-allow-headers"] = allow_headers
+            end
+          end
+
+          def set_access_control_max_age
+            access_control_max_age = ENV.fetch("FOOBARA_HTTP_RESPONSE_HEADER_ACCESS_CONTROL_MAX_AGE", nil)
+            if access_control_max_age
+              request.response_headers["access-control-max-age"] = access_control_max_age
+            end
+          end
         end
       end
     end

data/src/http/request.rb

@@ -93,7 +93,9 @@ module Foobara
         end
 
         def set_action_and_command_name
-          @action, @full_command_name = prefixless_path[1..].split("/")
+          @action, *full_command_name = prefixless_path[1..].split("/")
+
+          @full_command_name = full_command_name.join("::").gsub("/", "::")
         end
 
         def prefixless_path

data/src/http/response_mutators/set_header.rb

@@ -0,0 +1,38 @@
+module Foobara
+  module CommandConnectors
+    class Http < CommandConnector
+      class SetHeader < ResponseMutator
+        class << self
+          attr_accessor :header_name, :header_value
+
+          def for(header_name, header_value)
+            subclass = Class.new(self)
+
+            subclass.header_name = header_name
+            subclass.header_value = header_value
+
+            subclass
+          end
+        end
+
+        attr_writer :header_name, :header_value
+
+        def result_type_from(result_type)
+          result_type
+        end
+
+        def mutate(response)
+          response.add_header(header_name.to_s, header_value)
+        end
+
+        def header_name
+          @header_name ||= self.class.header_name
+        end
+
+        def header_value
+          @header_value ||= self.class.header_value
+        end
+      end
+    end
+  end
+end

data/src/http.rb

@@ -60,13 +60,11 @@ module Foobara
 
       def request_to_command(request)
         if request.method == "OPTIONS"
-          # TODO: this feels a bit hacky and like overkill...
-          return Foobara::CommandConnectors::Http::Commands::GetOptions.new
+          return Foobara::CommandConnectors::Http::Commands::GetOptions.new(request:)
         end
 
         command = super
 
-        # TODO: We should kill these case statements and require connecting these commands
         if request.action == "help"
           command.class.serializers = [Commands::Help::ResultSerializer]
         end
@@ -136,6 +134,12 @@ module Foobara
 
       def static_headers
         @static_headers ||= ENV.each_with_object({}) do |(key, value), headers|
+          next if %w[
+            FOOBARA_HTTP_RESPONSE_HEADER_ACCESS_CONTROL_ALLOW_HEADERS
+            FOOBARA_HTTP_RESPONSE_HEADER_ACCESS_CONTROL_ALLOW_METHODS
+            FOOBARA_HTTP_RESPONSE_HEADER_ACCESS_CONTROL_MAX_AGE
+          ].include?(key)
+
           match = key.match(/\AFOOBARA_HTTP_RESPONSE_HEADER_(.*)\z/)
 
           if match

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: foobara-http-command-connector
 version: !ruby/object:Gem::Version
-  version: 0.0.18
+  version: 0.0.20
 platform: ruby
 authors:
 - Miles Georgi
 bindir: bin
 cert_chain: []
-date: 2025-03-30 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: foobara
@@ -70,6 +70,7 @@ files:
 - src/http/response.rb
 - src/http/response_mutators/move_attribute_to_cookie.rb
 - src/http/response_mutators/move_attribute_to_header.rb
+- src/http/response_mutators/set_header.rb
 homepage: https://github.com/foobara/http-command-connector
 licenses:
 - Apache-2.0
@@ -93,7 +94,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.6
+rubygems_version: 3.6.7
 specification_version: 4
 summary: No description. Add one.
 test_files: []