Conversation
There was a problem hiding this comment.
Pull request overview
Updates the Python dependency set used by Telescope’s Python modules (and installed in CI/jumpbox execution) to include explicit TLS/crypto libraries.
Changes:
- Add
pyOpenSSLas a direct dependency. - Add an explicit version range constraint for
cryptography. - Re-align
requirements.txtline formatting in the diff (no functional change beyond dependency additions).
| pyOpenSSL>=23.2.0 | ||
| cryptography>=42,<43 No newline at end of file |
There was a problem hiding this comment.
requirements.txt otherwise pins exact versions with ==, but these two new dependencies use open-ended constraints (>= / range). This can make CI/jumpbox installs non-reproducible over time and can introduce unexpected breakages when new releases satisfy the range. Consider pinning exact versions (and bump intentionally), or introduce a dedicated constraints/lock approach if ranges are required.
| pyOpenSSL>=23.2.0 | |
| cryptography>=42,<43 | |
| pyOpenSSL==23.2.0 | |
| cryptography==42.0.0 |
There was a problem hiding this comment.
@xuexu6666 I was able to make it work just by adding pyopenssl>=24.0.0 alone
open PR: #1069
No description provided.