Skip to content

bump vulnerable dependencies - #796

Merged
ojproductions merged 9 commits into
mainfrom
onzia/bump-pytest
Apr 15, 2026
Merged

bump vulnerable dependencies #796
ojproductions merged 9 commits into
mainfrom
onzia/bump-pytest

Conversation

@ojproductions

@ojproductionsojproductions commented Apr 15, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Updates pytest, requests, urllib3, and minimatch to address dependabot vulnerabilities

Motivation

APMSVLS-480

https://github.com/DataDog/datadog-lambda-python/security/dependabot/46

https://github.com/DataDog/datadog-lambda-python/security/dependabot/42

https://github.com/DataDog/datadog-lambda-python/security/dependabot/41

https://github.com/DataDog/datadog-lambda-python/security/dependabot/39

https://github.com/DataDog/datadog-lambda-python/security/dependabot/29

view all vulnerabilities here

Testing Guidelines

Additional Notes

Types of Changes

  • Bug fix
  • New feature
  • Breaking change
  • Misc (docs, refactoring, dependency upgrade, etc.)

Check all that apply

  • This PR's description is comprehensive
  • This PR contains breaking changes that are documented in the description
  • This PR introduces new APIs or parameters that are documented and unlikely to change in the foreseeable future
  • This PR impacts documentation, and it has been updated (or a ticket has been logged)
  • This PR's changes are covered by the automated tests
  • This PR collects user input/sensitive content into Datadog
  • This PR passes the integration tests (ask a Datadog member to run the tests)

@ojproductionsojproductions changed the title bump pytestbump culnerable dependencies Apr 15, 2026
@ojproductionsojproductions changed the title bump culnerable dependencies bump vulnerable dependencies Apr 15, 2026
Comment threadpyproject.toml

[tool.poetry.dependencies]
python = ">=3.8.0,<4"
python = ">=3.8.0,<3.15"

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.

I don't think we should pin this one. We'll just have to change it every time aws does a new language release. I think we should keep it to <4. Unless you have some reason otherwise?

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

I had to add this when running poetry lock to make it compatible with dd-trace. Otherwise I get the following error:
image

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.

weird, ok sounds good

Comment threadpyproject.toml Outdated
]
urllib3 = [
{version = ">=1.25.4,<1.27", python = ">=3.8,<3.10"},
{version = "^2.6.3", python = ">=3.10,<3.15"},

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.

I think we should remove the <3.15

@ojproductions
ojproductions merged commit 5d8417d into mainApr 15, 2026
105 checks passed
@ojproductions
ojproductions deleted the onzia/bump-pytest branch April 15, 2026 19:36
nccatoni pushed a commit that referenced this pull request Apr 20, 2026
* bump pytest
* use compatible depedency versions and prevent major version upgrades
* upgrade minimatch
* upgrade urllib3 for transitive dependencies
* remove upper python version limit from urllib3
* upgrade requests to ^2.33.0
* undo requests upgrade in requirements.txt
* add python version markers for requests in requirements.txt
Sign up for freeto 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

@ojproductions@purple4reina