Skip to content

Add the param "verify" to support choose if SSL verify on ChatCompletion.create - #562

Closed
lishukan wants to merge 2 commits into
openai:mainfrom
lishukan:add_https_verify
Closed

lishukan wants to merge 2 commits into
openai:mainfrom
lishukan:add_https_verify

Conversation

@lishukan

@lishukan lishukan commented Aug 4, 2023 •

Copy link
Copy Markdown

This commit is relate to #561

This submission is to enable ChatCompletion.create to choose whether to perform SSL verification on its own. Now the default value for this parameter is None, which is consistent with the default value for “verify” in the requests module. Through this submission, when you want to ignore SSL verification, you can directly specify verify=False in the create method to avoid it.

the "verify" param in requests.request
image

my key change point
image

@lishukan lishukan changed the title Add the param "verfiy" to support choose if SSL verify on ChatCompletion.create Add the param "verify" to support choose if SSL verify on ChatCompletion.create Aug 4, 2023
@lishukan

lishukan commented Sep 5, 2023

Copy link
Copy Markdown
Author

@q-x-iao q-x-iao thanks your advise ,i 'll close this PR now

@lishukan lishukan closed this Sep 5, 2023
safa0 pushed a commit to safa0/openai-agents-python that referenced this pull request Apr 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant