Skip to content

fix(client-python): security updates (#367) - #368

Open
Ferdinand (guzmud) wants to merge 2 commits into
mainfrom
fix/365-security-updates
Open

Ferdinand (guzmud) wants to merge 2 commits into
mainfrom
fix/365-security-updates

Conversation

@guzmud

@guzmud Ferdinand (guzmud) commented Oct 21, 2026 •

Copy link
Copy Markdown
Member

Proposed changes

Testing Instructions

  1. Step-by-step how to test
  2. Environment or config notes

Related issues

Checklist

  • I consider the submitted work as finished
  • I tested the code for its functionality
  • I wrote test cases for the relevant uses case
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality
  • For bug fix -> I implemented a test that covers the bug

Further comments

Copilot AI balanced review requested due to automatic review settings October 21, 2026 13:31
@github-actions github-actions Bot added the filigran team Item from the Filigran team. label Oct 21, 2026
@guzmud Ferdinand (guzmud) changed the title fix(client-python): security updates fix(client-python): security updates (#367) Oct 21, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

The unsupported uv self-update command stops the coverage job before tests run.

1 open issue
What changed in this PR

Updates Python client dependencies and CI security tooling for #367.

Changes:

  • Adds a minimum certifi version.
  • Adds optional Bandit security scanning.
  • Updates pip and certificate packages in CI workflows.
File Description
pyproject.toml Requires certifi>=2026.10.19.
.github/​workflows/​sats-analysis.yml Adds optional Bandit scanning.
.github/​workflows/​release.yml Upgrades pip and certifi before release dependencies.
.github/​workflows/​coverage.yml Adds pip, certifi, and uv update commands.
.circleci/​config.yml Upgrades pip and certifi across jobs.

Review effort: Balanced

Comment thread .github/workflows/coverage.yml Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔵 Needs a closer look

Availability of the specified upstream versions and compatibility with the CI environments remain unverified.

No open issues 🎉

1 resolved since last review

Review effort: Balanced

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔵 Needs a closer look

Dependency and tooling compatibility needs successful validation across the affected GitHub Actions and CircleCI environments.

No open issues 🎉

Review effort: Balanced

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

filigran team Item from the Filigran team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(client-python): security updates

3 participants