Added http_client option to the BaseOpenAIClientConfiguration class

[avinashmihani]

Added http_client option to the BaseOpenAIClientConfiguration class, to support SSL verification option in model_client intiation.

Why are these changes needed?

In v0.2.x the agents to interact with LLM, was expecting LLM_CONFIG, which had an option to skip the SSL verification. Now in v0.4.x, the model client initiation does not have an option for skipping verification.
With this change, v0.4 as well will allow http_client option in model_client initiation

Related issue number

Closes #5795

Checks

• I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://github.com/microsoft/autogen/blob/main/CONTRIBUTING.md to build and test documentation locally.
• I've added tests (if relevant) corresponding to the changes introduced in this PR.
• I've made sure all auto checks have passed.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.58%. Comparing base (97536af) to head (ea79d65).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #5811   +/-   ##
=======================================
  Coverage   75.58%   75.58%           
=======================================
  Files         189      189           
  Lines       12669    12671    +2     
=======================================
+ Hits         9576     9578    +2     
  Misses       3093     3093           

Flag	Coverage Δ
unittests	75.58% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ekzhu]

@avinashmihani actually I just realized the http_client should already passed to AsyncOpenAI constructor without this change.

Does this change actually resolves #5795 ?

[avinashmihani]

@avinashmihani actually I just realized the http_client should already passed to AsyncOpenAI constructor without this change.

Does this change actually resolves #5795 ?

@ekzhu Yes, this change worked. I tested in my local. With this change, I am able to update SSL verification value in v0.4.x.

[ekzhu]

The problem though is that CreateArguments is a typed dictionary so it can only contain typed annotation. So we can't use the httpx.AsyncClient type. Perhaps we use Any here? @jackgerrits what's your thought?

[jackgerrits]

You can't put anything into that dictionary that is not json serializable so this is not going to work. This is required for the component config feature.

If a python object must be passed it must be done directly via the constructor and so it wont be supported via config. If it is necesasary to support this argument we will need to strip it out of the exported config dictionary in _to_config