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.

Sign up for GitHub

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

Jump to bottom

feat: use transport package #228

Draft

shyim wants to merge 1 commit into opensearch-project:main from shyim:use-transport-package

Collaborator

shyim commented Sep 8, 2024 •

edited

Loading

Description

Fix all tests
Add Sigv4 again
Allow setting custom ssl cert, disable certificate verification
Fix async http requests?


          feat: use transport package

5be2b8d

Member

dblock commented Sep 8, 2024

Very nice. I'd start building up UPGRADING.md like we have in https://github.com/opensearch-project/opensearch-py/blob/main/UPGRADING.md to note all the backwards incompatible changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

VachaShah Awaiting requested review from VachaShah VachaShah will be requested when the pull request is marked ready for review VachaShah is a code owner

dblock Awaiting requested review from dblock dblock will be requested when the pull request is marked ready for review dblock is a code owner

harshavamsi Awaiting requested review from harshavamsi harshavamsi will be requested when the pull request is marked ready for review harshavamsi is a code owner

saimedhi Awaiting requested review from saimedhi saimedhi will be requested when the pull request is marked ready for review saimedhi is a code owner

At least 1 approving review is required to merge this pull request.

Labels

None yet