Merge extra headers into request headers for more flexibility

[deikka]

The http.rb file now merges extra headers into the request headers, allowing for more flexibility in the headers that can be sent with each request. This change is particularly useful when there are headers specific to a certain request that are not included in the default headers.

OpenAI::Client.new.chat(
      parameters: {
        model: 'gpt-3.5-turbo',
        messages: Message.for_openai(trip.messages),
        temperature: 0.1,
        stream: stream_proc(message:),
        extra_headers: {
          'Helicone-Property-Conversation': 'conversation-123'
        }
      }
    )
  end

The http_spec.rb file now includes a test to ensure that these extra headers are correctly included in the chat request.

Related issue

All Submissions:

• [x] Have you followed the guidelines in our Contributing document?
• [x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
• [x] Have you added an explanation of what your changes do and why you'd like us to include them?

[alexrudall]

Thanks for your work on this @deikka! It ended up getting resolved in a slightly different way: See here

[deikka]

Thank you, @alexrudall I couldn't give it the care it deserved ;) I like how you did it.

[alexrudall]

Thank you, @alexrudall I couldn't give it the care it deserved ;) I like how you did it.

No worries! That would be @Haegin, I just merge things ;)