opensearch-project / opensearch-py

Python Client for OpenSearch
https://opensearch.org/docs/latest/clients/python/
Apache License 2.0
338 stars 170 forks source link

Updated dependencies, generated API. #793

Closed dblock closed 1 month ago

dblock commented 1 month ago

Description

Issues Resolved

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. For more information on following Developer Certificate of Origin and signing off your commits, please check here.

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 57.03704% with 116 lines in your changes missing coverage. Please review.

Project coverage is 70.25%. Comparing base (ba715b9) to head (f7a7397). Report is 45 commits behind head on main.

Files Patch % Lines
opensearchpy/_async/plugins/sql.py 48.38% 16 Missing :warning:
opensearchpy/plugins/sql.py 48.38% 16 Missing :warning:
opensearchpy/_async/plugins/observability.py 56.66% 13 Missing :warning:
opensearchpy/plugins/observability.py 56.66% 13 Missing :warning:
opensearchpy/_async/plugins/ppl.py 52.38% 10 Missing :warning:
opensearchpy/plugins/ppl.py 52.38% 10 Missing :warning:
opensearchpy/_async/plugins/query.py 59.09% 9 Missing :warning:
opensearchpy/plugins/query.py 59.09% 9 Missing :warning:
opensearchpy/_async/plugins/ml.py 40.00% 6 Missing :warning:
opensearchpy/plugins/ml.py 40.00% 6 Missing :warning:
... and 2 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #793 +/- ## ========================================== - Coverage 71.95% 70.25% -1.71% ========================================== Files 91 113 +22 Lines 8001 8881 +880 ========================================== + Hits 5757 6239 +482 - Misses 2244 2642 +398 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

dblock commented 1 month ago

Maybe @saimedhi @VachaShah can review?

saimedhi commented 1 month ago
saimedhi commented 1 month ago
  • looks good to me.
  • Going through the CVEs to confirm if below are right version bumps
install_requires = [
    'urllib3>=1.26.19,<1.27 ; python_version < "3.10"',
    'urllib3>=1.26.19,!=2.2.0,<3 ; python_version >= "3.10"',
    "requests>=2.32.0, <3.0.0",
    "python-dateutil",
    "certifi>=2024.07.04",
    "Events",
]
dblock commented 1 month ago
  • 'urllib3>=1.26.19,!=2.2.0,<3 ; python_version >= "3.10"', here 2.2.1 should also be removed as per affected versions here GHSA-34jh-p97f-mpxf

Done.