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

[CHORE] dependency pinning #168

Merged
merged 3 commits into from
Nov 27, 2024
Merged

[CHORE] dependency pinning #168

merged 3 commits into from
Nov 27, 2024

Conversation

teej
Copy link
Collaborator

@teej teej commented Nov 27, 2024

No description provided.

@teej teej temporarily deployed to snowflake-gcp-standard November 27, 2024 19:56 — with GitHub Actions Inactive
@teej teej temporarily deployed to snowflake-aws-standard November 27, 2024 19:56 — with GitHub Actions Inactive
@teej teej temporarily deployed to snowflake-aws-enterprise November 27, 2024 19:56 — with GitHub Actions Inactive
@teej teej temporarily deployed to snowflake-aws-business-critical November 27, 2024 19:56 — with GitHub Actions Inactive
@teej teej temporarily deployed to snowflake-azure-standard November 27, 2024 19:56 — with GitHub Actions Inactive
Copy link

codecov bot commented Nov 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Files with missing lines Coverage Δ
setup.py 0.00% <ø> (ø)

... and 2 files with indirect coverage changes

@teej teej temporarily deployed to snowflake-gcp-standard November 27, 2024 20:05 — with GitHub Actions Inactive
@teej teej temporarily deployed to snowflake-aws-standard November 27, 2024 20:05 — with GitHub Actions Inactive
@teej teej temporarily deployed to snowflake-aws-enterprise November 27, 2024 20:05 — with GitHub Actions Inactive
@teej teej temporarily deployed to snowflake-aws-business-critical November 27, 2024 20:05 — with GitHub Actions Inactive
@teej teej temporarily deployed to snowflake-azure-standard November 27, 2024 20:05 — with GitHub Actions Inactive
@teej teej temporarily deployed to snowflake-gcp-standard November 27, 2024 20:08 — with GitHub Actions Inactive
@teej teej temporarily deployed to snowflake-aws-standard November 27, 2024 20:08 — with GitHub Actions Inactive
@teej teej had a problem deploying to snowflake-aws-enterprise November 27, 2024 20:08 — with GitHub Actions Failure
@teej teej temporarily deployed to snowflake-aws-business-critical November 27, 2024 20:08 — with GitHub Actions Inactive
@teej teej temporarily deployed to snowflake-azure-standard November 27, 2024 20:08 — with GitHub Actions Inactive
@teej teej merged commit db09038 into main Nov 27, 2024
8 of 9 checks passed
@teej
Copy link
Collaborator Author

teej commented Nov 27, 2024

This change adds the dependency "pyOpenSSL>=22.1.0". This is necessary due to a change in the Python cryptography library getting an update today (pypi) and a bit of software supply chain issues.

  • Titan relies on the snowflake-connector-python library.
  • That library relies on PyOpenSSL.
  • That library relies on cryptography.
  • The latter had an update that changed an API.
  • PyOpenSSL has anticipated this and was fixed many versions ago (fixed in 22.1.0, current 24.2.1).
  • Unfortunately, snowflake-connector-python specifies an overly-broad set of acceptable versions pyOpenSSL>=16.2.0,<25.0.0 (source) despite the fact they only test on recent versions pyOpenSSL==24.2.1 (source).

See snowflakedb/snowflake-connector-python#2109 for more info.

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.

1 participant