Skip to content
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.

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

Security updates #6387

Merged
merged 8 commits into from May 10, 2024
Merged

Security updates #6387

merged 8 commits into from May 10, 2024

Conversation

piiq
Copy link
Contributor

@piiq piiq commented May 10, 2024

This PR is for addressing most of the security alerts raised by Dependabot

bumps aiohttp, python-multipart and urllib3

only lock files (and core/pyproject.toml) changed

this should remove a few hundred alerts

@piiq piiq added the security label May 10, 2024
@piiq piiq self-assigned this May 10, 2024
@github-actions github-actions bot added enhancement Enhancement platform OpenBB Platform v4 PRs for v4 labels May 10, 2024
@IgorWounds
Copy link
Contributor

@piiq I ran all unit and integration tests. Things are looking good. I've noticed that some integration tests that are using the TMX provider fail:

  • test_fixedincome_corporate_bond_prices
  • test_fixedincome_government_treasury_prices

This provider uses two dependencies that might be linked to the change:

  • aiohttp-client-cache = "^0.10.0"
  • aiosqlite = "^0.19.0"

@piiq
Copy link
Contributor Author

piiq commented May 10, 2024

@IgorWounds I've bumped those and the integration tests for tmx are passing on my side.
Can you check please?
image

@piiq
Copy link
Contributor Author

piiq commented May 10, 2024

I am done with commits in this PR. I will create another one to remove python-jose and bump cryptography and starlette

@piiq piiq added the P2 label May 10, 2024
@IgorWounds IgorWounds added this pull request to the merge queue May 10, 2024
Merged via the queue into develop with commit ac3f10a May 10, 2024
10 checks passed
@IgorWounds IgorWounds deleted the feature/improved-security branch May 11, 2024 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement P2 platform OpenBB Platform security v4 PRs for v4
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants