Skip to content

python(fix): allow patched filelock and drop pyOpenSSL - #855

Merged
alexluck-sift merged 3 commits into
mainfrom
al/CVE-2026-112940
Oct 26, 2026
Merged

alexluck-sift merged 3 commits into
mainfrom
al/CVE-2026-112940

Conversation

@alexluck-sift

Copy link
Copy Markdown
Collaborator

Summary

Two dependency pins kept customer installs on releases with published vulnerabilities, even when fixed releases existed. This PR removes both blocks.

filelock

  • The pin moves from ~=3.15 to >=3.15,<5, so Python 3.11+ resolves filelock 4.2.1 or later (GHSA-v4x8-3wr5-3wq8, GHSA-9vhv-jxww-xfm5, GHSA-567q-5qvw-pf92). The SDK only uses AsyncFileLock, not the affected ReadWriteLock or SoftFileLock.
  • Python 3.10 stays on a vulnerable filelock: 4.2.1 requires Python 3.11.

pyOpenSSL removal

  • cert_via_openssl now converts the system CA certificates to PEM with ssl.DER_cert_to_PEM_cert instead of pyOpenSSL.
  • The openssl extra is removed. Its pyOpenSSL<24 pin capped cryptography below 42, which has several high-severity advisories. pip install sift-stack-py[openssl] still succeeds, with a warning that the extra does not exist.

Verification

  • On a macOS system CA store (152 certificates), the new stdlib output is byte-identical to the old pyOpenSSL output.
  • Built gRPC credentials and REST adapters through sift_client and sift_py with cert_via_openssl=True and no pyOpenSSL installed.

Widen the filelock pin to <5 so installs on Python 3.11+ resolve
filelock 4.2.1 or later. Replace pyOpenSSL with the standard library
ssl module for cert_via_openssl and remove the openssl extra, which
capped cryptography below 42.
@github-actions

github-actions Bot commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

Python docs preview: https://sift-stack.github.io/sift/python/pr-855/

Deployed from 86d5540. The link may take up to a minute to become live as GitHub Pages propagates.

@alexluck-sift
alexluck-sift requested review from marc-sift, solidiquis and wei-sift and removed request for wei-sift October 26, 2026 19:28
@alexluck-sift
alexluck-sift marked this pull request as ready for review October 26, 2026 19:31

@wei-sift wei-sift left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@alexluck-sift
alexluck-sift merged commit 885153b into main Oct 26, 2026
27 checks passed
@alexluck-sift
alexluck-sift deleted the al/CVE-2026-112940 branch October 26, 2026 20:15
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