Skip to content

security: pin transitive dependencies (pyasn1, mako)#7734

Open
wtfiwtz wants to merge 3 commits into
getredash:masterfrom
orchestrated-io:security/transitive-dependency-pins
Open

security: pin transitive dependencies (pyasn1, mako)#7734
wtfiwtz wants to merge 3 commits into
getredash:masterfrom
orchestrated-io:security/transitive-dependency-pins

Conversation

@wtfiwtz
Copy link
Copy Markdown

@wtfiwtz wtfiwtz commented Jun 2, 2026

What type of PR is this?

  • Refactor

Description

Pin transitive dependencies to address scanner-detected vulnerabilities:

  • pyasn1: 0.6.3
  • mako: 1.3.12

These packages are transitive dependencies (pulled in by other packages) that have known vulnerabilities. Pinning them ensures the secure versions are used.

Note: pynacl 1.6.2 would also be beneficial but requires cffi >=2.0.0, which is only compatible with snowflake-connector-python 4.x. That dependency chain is addressed in PR #7729.

How is this tested?

  • Unit tests (pytest)
  • Manually

Related Tickets & Documents

Split from orchestrated-io/redash#7719 per @zachliu's review feedback to separate transitive dependency pins from direct dependency upgrades.

Part of the security vulnerability remediation work tracked in #7711.

Note

This PR is marked as draft until manual testing is complete.

Made with Cursor

Pin transitive dependencies to address scanner-detected vulnerabilities:

- pyasn1: 0.6.3
- mako: 1.3.12

These packages are transitive dependencies (pulled in by other packages)
that have known vulnerabilities. Pinning them ensures the secure versions
are used.

Note: pynacl 1.6.2 would also be beneficial but requires cffi >=2.0.0,
which is only compatible with snowflake-connector-python 4.x (upgraded
in a separate PR).

Related: split from orchestrated-io/redash#7719
Co-authored-by: Cursor <cursoragent@cursor.com>
@wtfiwtz wtfiwtz marked this pull request as ready for review June 3, 2026 22:09
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 4 files

Re-trigger cubic

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