Skip to content
This repository was archived by the owner on Dec 5, 2025. It is now read-only.

[deps]: loose requests required versions#859

Merged
flavienSindou merged 2 commits intomasterfrom
bug/857-requests-requirements-too-strict
Mar 12, 2025
Merged

[deps]: loose requests required versions#859
flavienSindou merged 2 commits intomasterfrom
bug/857-requests-requirements-too-strict

Conversation

@flavienSindou
Copy link
Contributor

@flavienSindou flavienSindou commented Mar 7, 2025

Proposed changes

  • Loose requests required versions in dependencies

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

Further comments

From Changelog (consulted on March 6, 2025) last Security issue was patch in 2.32..0. So we tested from this version to latest one with python versions 3.8 -> 3.12.

Using the tox tool, and based on successful package installation with pip, vulnerability check with pip-audit and tests performed with pytest, we can conclude that we are, at a minimum, compatible with requests versions [2.32.0 to 2.32.3] when using Python [3.8 to 3.12].

Investigation Notion Page : https://www.notion.so/filigran/Loose-Requests-required-versions-in-opencti-python-client-1ae8fce17f2a809c9eebfec93400b1c4?pvs=4

@flavienSindou flavienSindou self-assigned this Mar 7, 2025
@flavienSindou flavienSindou linked an issue Mar 7, 2025 that may be closed by this pull request
@flavienSindou flavienSindou added bug use for describing something not working as expected filigran team use to identify PR from the Filigran team labels Mar 7, 2025
@flavienSindou flavienSindou merged commit 0bfb741 into master Mar 12, 2025
6 checks passed
@flavienSindou flavienSindou deleted the bug/857-requests-requirements-too-strict branch March 12, 2025 16:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug use for describing something not working as expected filigran team use to identify PR from the Filigran team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Requests requirements too strict

2 participants