Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add parameter ssl = false #2

Merged
merged 2 commits into from
Nov 28, 2024
Merged

add parameter ssl = false #2

merged 2 commits into from
Nov 28, 2024

Conversation

rubenLeapFinancial
Copy link
Contributor

This pull request includes changes to the celai_chatwoot/connector/bot_utils.py and celai_chatwoot/connector/msg_utils.py files to disable SSL verification in aiohttp.ClientSession for various asynchronous functions. This update aims to simplify the HTTP requests by bypassing SSL certificate verification.

Updates to bot_utils.py:

  • Modified list_agent_bots, create_agent_bot, delete_agent_bot, get_agent_bot, update_agent_bot, and assign_bot_to_inbox functions to use aiohttp.ClientSession(connector=aiohttp.TCPConnector(ssl=False)) for HTTP requests. [1] [2] [3] [4] [5] [6]

Updates to msg_utils.py:

  • Modified __build_content_image, __build_content_audio, send_text_message, and send_attachment functions to use aiohttp.ClientSession(connector=aiohttp.TCPConnector(ssl=False)) for HTTP requests. [1] [2] [3] [4]

@rubenLeapFinancial
Copy link
Contributor Author

This pull request introduces changes to support SSL configuration in the aiohttp.ClientSession for various methods in the celai_chatwoot package. The primary focus is on adding an ssl parameter to the constructors and using this parameter to configure the aiohttp.TCPConnector.

Key changes include:

SSL Configuration Support:

  • celai_chatwoot/connector/bot_utils.py: Added an ssl parameter to the constructor and updated aiohttp.ClientSession usage to include aiohttp.TCPConnector(ssl=self.ssl) for methods like list_agent_bots, create_agent_bot, delete_agent_bot, get_agent_bot, update_agent_bot, and assign_bot_to_inbox. [1] [2] [3] [4] [5] [6]

  • celai_chatwoot/connector/msg_utils.py: Added an ssl parameter to the constructor and updated aiohttp.ClientSession usage to include aiohttp.TCPConnector(ssl=self.ssl) for methods like __build_content_image, __build_content_audio, send_text_message, and send_attachment. [1] [2] [3] [4] [5]

  • celai_chatwoot/connector/woo_connector.py: Added an ssl parameter to the constructor and updated ChatwootMessages and ChatwootAgentsBots instantiation to pass the ssl parameter for methods like send_text_message, send_image_message, send_audio_message, and update_bot. [1] [2] [3] [4] [5] [6]

@alejamp alejamp merged commit d8f6f57 into cel-ai:main Nov 28, 2024
1 check passed
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.

2 participants