Merge pull request alexrudall#473 from drnic/remove-warning-no-org

alexrudall · web-flow · commit e06565a2fa67 · 2024-06-10T12:43:46.000+01:00
If being used for Groq, skip OpenAI-Organization header
diff --git a/.gitignore b/.gitignore
@@ -10,11 +10,6 @@
 /test/tmp/
 /test/version_tmp/
 /tmp/
-/.bundle/
-/.yardoc
-/_yardoc/
-/doc/
-
 
 # Used by dotenv library to load environment variables.
 .env
diff --git a/lib/openai/http_headers.rb b/lib/openai/http_headers.rb
@@ -19,7 +19,7 @@ def openai_headers
         "Content-Type" => "application/json",
         "Authorization" => "Bearer #{@access_token}",
         "OpenAI-Organization" => @organization_id
-      }
+      }.compact
     end
 
     def azure_headers
diff --git a/spec/openai/client/http_spec.rb b/spec/openai/client/http_spec.rb
@@ -263,7 +263,7 @@
 
     it {
       expect(headers).to eq({ "Authorization" => "Bearer #{OpenAI.configuration.access_token}",
-                              "Content-Type" => "application/json", "OpenAI-Organization" => nil })
+                              "Content-Type" => "application/json" })
     }
 
     describe "with Azure" do

Original file line number	Diff line number	Diff line change
`@@ -263,7 +263,7 @@`
`263`	`263`
`264`	`264`	`it {`
`265`	`265`	`expect(headers).to eq({ "Authorization" => "Bearer #{OpenAI.configuration.access_token}",`
`266`		`- "Content-Type" => "application/json", "OpenAI-Organization" => nil })`
	`266`	`+ "Content-Type" => "application/json" })`
`267`	`267`	`}`
`268`	`268`
`269`	`269`	`describe "with Azure" do`