Skip to content

[SYNPY-1906] fix: resolve security vulnerabilities - #1441

Open
linglp wants to merge 5 commits into
developfrom
fix-security
Open

[SYNPY-1906] fix: resolve security vulnerabilities#1441
linglp wants to merge 5 commits into
developfrom
fix-security

Conversation

@linglp

@linglp linglp commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Problem:

See: https://github.com/Sage-Bionetworks/synapsePythonClient/security/dependabot
Screenshot 2026-08-06 at 11 03 49 PM

Solution:

  • Relock the pipfile
  • Updated setup.cfg to have cryptography >= 50.0.0 and pytest ~= 9.0.3

Comment thread Pipfile.lock
"markers": "python_version >= '3.10'",
"version": "==8.3.1"
"markers": "python_version >= '3.9' and python_full_version not in '3.9.0, 3.9.1'",
"version": "==50.0.0"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Relock the lock file to get the patch version 50.0.0: https://github.com/Sage-Bionetworks/synapsePythonClient/security/dependabot/57

Comment thread setup.cfg
cryptography>=48.0.1
tests_require =
pytest~=8.2.0
pytest~=9.0.3

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@linglp linglp changed the title Fix: resolve security vulnerabilities [SYNPY-1906]fix: resolve security vulnerabilities Aug 7, 2026
@linglp linglp changed the title [SYNPY-1906]fix: resolve security vulnerabilities [SYNPY-1906]resolve security vulnerabilities Aug 7, 2026
@linglp linglp changed the title [SYNPY-1906]resolve security vulnerabilities [SYNPY-1906] fix: resolve security vulnerabilities Aug 7, 2026
@linglp
linglp marked this pull request as ready for review August 7, 2026 03:19
@linglp
linglp requested a review from a team as a code owner August 7, 2026 03:19
Copilot AI lite review requested due to automatic review settings August 7, 2026 03:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates Python package dependency constraints in setup.cfg to address Dependabot-reported security vulnerabilities in the Synapse Python Client packaging/test environment.

Changes:

  • Bumped cryptography minimum version to >=50.0.0.
  • Updated pytest version pin in tests_require and extras_require to ~=9.0.3.
Suppressed comments (2)

setup.cfg:82

  • PR description says pytest >= 9.0.3, but this extras pin uses pytest~=9.0.3 (restricts to <9.1.0). Consider switching to pytest>=9.0.3 here as well (or align the PR description if the pin is intentional).
dev =
    pytest~=9.0.3
    pytest-mock>=3.0,<4.0

setup.cfg:98

  • PR description says pytest >= 9.0.3, but this extras pin uses pytest~=9.0.3 (restricts to <9.1.0). Consider switching to pytest>=9.0.3 here as well (or align the PR description if the pin is intentional).
tests =
    pytest~=9.0.3
    pytest-mock>=3.0,<4.0

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread setup.cfg
Comment on lines 66 to 68
tests_require =
pytest~=8.2.0
pytest~=9.0.3
pytest-mock>=3.0,<4.0
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.

2 participants