Skip to content

fix(deps): audit the server extra and dev group and resolve their findings #144

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.

Sign up for GitHub

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

Merged
tkislan merged 5 commits into from
Oct 28, 2026
Merged

fix(deps): audit the server extra and dev group and resolve their findings #144

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
adb9b07
fix(ci): run pip-audit for the server extra and the dev group
tkislan Oct 26, 2026
a9f611d
fix(deps): resolve pip-audit findings in the server extra and dev group
tkislan Oct 26, 2026
e71e3ed
fix(ci): limit the new audit jobs' token to read-only contents
tkislan Oct 26, 2026
4d39d59
refactor(ci): run the pip-audit jobs as one matrix job
tkislan Oct 27, 2026
0b60f86
Merge remote-tracking branch 'origin/main' into tk/audit-server-dev-d…
tkislan Oct 27, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
61 changes: 18 additions & 43 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -473,10 +473,22 @@ jobs:
/tmp/smoke/bin/python -m pip install "${WHEEL_PATH}[server]"
/tmp/smoke/bin/python scripts/check_toolkit_imports.py

audit-prod:
name: Audit - Production
audit:
name: Audit - ${{ matrix.name }}
runs-on: ubuntu-latest
timeout-minutes: 3
permissions:
contents: read
strategy:
fail-fast: false
matrix:
include:
- name: Production
export-args: --without dev
- name: Server
export-args: --extras server
- name: Dev
export-args: --only dev
steps:
- uses: actions/checkout@08eba0b27e820071cde6df949e0beb9ba4906955 # v4
with:
Expand All @@ -499,13 +511,13 @@ jobs:
- name: Install poetry-plugin-export
run: poetry self add poetry-plugin-export

- name: Run audit for production dependencies
- name: Export dependencies
run: |
poetry export --without dev -f requirements.txt -o /tmp/requirements-prod.txt
poetry export ${{ matrix.export-args }} -f requirements.txt -o /tmp/requirements.txt

- uses: pypa/gh-action-pip-audit@f1c3022531130da3984dbdfb2786f6e1cba15556 # latest release (v1.1.0 doesn't contain `disable-pip`)
with:
inputs: /tmp/requirements-prod.txt
inputs: /tmp/requirements.txt
require-hashes: true
disable-pip: true
# PYSEC-2024-161 (pyarrow deserialization vulnerability)
Expand All @@ -519,44 +531,7 @@ jobs:
# PYSEC-2026-4114 (oauthlib PKCE code_verifier timing side-channel)
# * not exploitable here: affects server-side AuthorizationCodeGrant validation; databricks-sql-connector only uses the client side (WebApplicationClient)
# * fix is in oauthlib 4.0.0, but databricks-sql-connector (latest 4.6.0) caps oauthlib<4.0.0
ignore-vulns: &ignore-vulns |
ignore-vulns: |
PYSEC-2024-161
PYSEC-2026-113
PYSEC-2026-4114

audit-all:
name: Audit - All
runs-on: ubuntu-latest
timeout-minutes: 3
steps:
- uses: actions/checkout@08eba0b27e820071cde6df949e0beb9ba4906955 # v4
with:
persist-credentials: false

- name: Set up Python
id: setup-python
uses: actions/setup-python@a26af69be951a213d495a4c3e4e4022e16d87065 # v5
with:
python-version: '3.11'

- name: Install Poetry
uses: snok/install-poetry@76e04a911780d5b312d89783f7b1cd627778900a # v1
with:
version: ${{ env.POETRY_VERSION }}
virtualenvs-create: ${{ env.POETRY_VIRTUALENVS_CREATE }}
virtualenvs-in-project: ${{ env.POETRY_VIRTUALENVS_IN_PROJECT }}
installer-parallel: ${{ env.POETRY_INSTALLER_PARALLEL }}

- name: Install poetry-plugin-export
run: poetry self add poetry-plugin-export

- name: Run audit for all dependencies
run: |
poetry export -f requirements.txt -o /tmp/requirements-all.txt

- uses: pypa/gh-action-pip-audit@f1c3022531130da3984dbdfb2786f6e1cba15556 # latest release (v1.1.0 doesn't contain `disable-pip`)
with:
inputs: /tmp/requirements-all.txt
require-hashes: true
disable-pip: true
ignore-vulns: *ignore-vulns
Loading
Loading