chore(deps): bump the pip group across 2 directories with 4 updates#629
Open
dependabot[bot] wants to merge 3 commits into
Open
chore(deps): bump the pip group across 2 directories with 4 updates#629dependabot[bot] wants to merge 3 commits into
dependabot[bot] wants to merge 3 commits into
Conversation
Contributor
Author
LabelsThe following labels could not be found: Please fix the above issues or remove invalid values from |
Dependency ReviewThe following issues were found:
Vulnerabilitiesrequirements.txt
Only included vulnerabilities with severity high or higher. License Issuesrequirements.txt
src/500-application/506-ros2-connector/services/requirements.base.txt
OpenSSF Scorecard
Scanned Files
|
c55b0aa to
ae035ea
Compare
Collaborator
|
@dependabot rebase |
ae035ea to
ba9204b
Compare
This was referenced Jun 30, 2026
Closed
bindsi
pushed a commit
that referenced
this pull request
Jun 30, 2026
…642) ## Description The *Dependency Review* gate in the **PR Validation** workflow was failing on the open Dependabot pip-group update (#629). Its only high-severity finding was **ecdsa 0.19.2** (*GHSA-wj6h-64fc-37mp*, "Minerva timing attack on P-256"), a **new transitive dependency** introduced by the *checkov* 3.2.529 → 3.3.5 bump. The advisory has no fixed upstream version, so no dependency bump can satisfy `fail-on-severity: high`. This change adds a narrowly-scoped `allow-ghsas` entry for that single advisory to the *Dependency Review* step in [.github/workflows/pr-validation.yml](.github/workflows/pr-validation.yml), with inline comments documenting the rationale and a pointer to re-evaluate when *checkov* drops the *ecdsa* dependency. No other workflow settings change. ## Related Issue Fixes #641 ## Type of Change <!-- What type of change does this PR introduce? Mark relevant options with 'x' --> - [ ] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) - [ ] Blueprint modification or addition - [ ] Component modification or addition - [ ] Documentation update - [x] CI/CD pipeline change - [ ] Other (please describe): ## Implementation Details Added `allow-ghsas: GHSA-wj6h-64fc-37mp` to the `actions/dependency-review-action` step within the `dependency-scan` job. The `fail-on-severity: high`, `warn-on-openssf-scorecard-level: 3`, and `comment-summary-in-pr: on-failure` settings are unchanged, so the gate still fails on every other high-or-higher advisory. Two preceding comments record that *ecdsa* is affected by a side-channel with no upstream fix, that it is transitive via *checkov*, and that the suppression should be revisited when that dependency is dropped. ## Testing Performed <!-- Describe the testing you have performed or plan to perform --> - [ ] Terraform plan/apply - [ ] Blueprint deployment test - [ ] Unit tests - [ ] Integration tests - [ ] Bug fix includes regression test (see [Test Policy](docs/contributing/testing-validation.md)) - [ ] Manual validation - [ ] Other: ## Validation Steps 1. Re-run (or rebase) Dependabot PR #629 and confirm the **Dependency Scan** check now passes. 2. Confirm an unrelated high-severity advisory still fails the gate (no broadening of the suppression beyond *GHSA-wj6h-64fc-37mp*). ## Checklist <!-- Mark relevant options with 'x' --> - [ ] I have updated the documentation accordingly - [ ] I have added tests to cover my changes - [ ] All new and existing tests passed - [ ] I have run `terraform fmt` on all Terraform code - [ ] I have run `terraform validate` on all Terraform code - [ ] I have run `az bicep format` on all Bicep code - [ ] I have run `az bicep build` to validate all Bicep code - [x] I have checked for any sensitive data/tokens that should not be committed - [ ] Lint checks pass (run applicable linters for changed file types) ## Security Review <!-- Required for PRs touching security-sensitive paths: - SECURITY.md - src/000-cloud/010-security-identity/ - deploy/ PRs modifying these paths require the `security-reviewed` label before merge. --> - [x] No credentials, secrets, or tokens are hardcoded or logged - [x] RBAC and identity changes follow least-privilege principles - [x] No new network exposure or public endpoints introduced without justification - [x] Dependency additions or updates have been reviewed for known vulnerabilities - [ ] Container image changes use pinned digests or SHA references ## Additional Notes This intentionally suppresses one known advisory (*GHSA-wj6h-64fc-37mp*) because no patched *ecdsa* release exists; the python-ecdsa maintainers treat the Minerva side-channel as out of scope. The suppression mirrors the existing transitive-with-no-fix pattern documented in [osv-scanner.toml](osv-scanner.toml) (which feeds osv-scanner rather than the Dependency Review action). Re-evaluate and remove the entry once *checkov* no longer pulls in *ecdsa*. ## Screenshots (if applicable) <!-- Add screenshots to show the changes, if applicable -->
Collaborator
|
@dependabot rebase |
Bumps the pip group with 1 update in the / directory: [checkov](https://github.com/bridgecrewio/checkov). Bumps the pip group with 3 updates in the /src/500-application/506-ros2-connector directory: [numpy](https://github.com/numpy/numpy), [pytest](https://github.com/pytest-dev/pytest) and [pytest-asyncio](https://github.com/pytest-dev/pytest-asyncio). Updates `checkov` from 3.2.529 to 3.3.6 - [Release notes](https://github.com/bridgecrewio/checkov/releases) - [Changelog](https://github.com/bridgecrewio/checkov/blob/main/CHANGELOG.md) - [Commits](bridgecrewio/checkov@3.2.529...3.3.6) Updates `numpy` from 2.4.6 to 2.5.0 - [Release notes](https://github.com/numpy/numpy/releases) - [Changelog](https://github.com/numpy/numpy/blob/main/doc/RELEASE_WALKTHROUGH.rst) - [Commits](numpy/numpy@v2.4.6...v2.5.0) Updates `pytest` from 9.0.3 to 9.1.1 - [Release notes](https://github.com/pytest-dev/pytest/releases) - [Changelog](https://github.com/pytest-dev/pytest/blob/main/CHANGELOG.rst) - [Commits](pytest-dev/pytest@9.0.3...9.1.1) Updates `pytest-asyncio` from 1.3.0 to 1.4.0 - [Release notes](https://github.com/pytest-dev/pytest-asyncio/releases) - [Commits](pytest-dev/pytest-asyncio@v1.3.0...v1.4.0) --- updated-dependencies: - dependency-name: checkov dependency-version: 3.3.1 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: pip - dependency-name: numpy dependency-version: 2.5.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: pip - dependency-name: pytest dependency-version: 9.1.1 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: pip - dependency-name: pytest-asyncio dependency-version: 1.4.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: pip ... Signed-off-by: dependabot[bot] <[email protected]>
ba9204b to
0c279fc
Compare
ecdsa 0.19.2 is a new transitive dependency pulled in by checkov 3.3.6. The package is unmaintained with no upstream fix for the Minerva timing attack (P-256). Already allow-listed in dependency-review-action; adding consistent coverage in grype config. Re-evaluate when checkov drops ecdsa. Refs: #641
8 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bumps the pip group with 1 update in the / directory: checkov.
Bumps the pip group with 3 updates in the /src/500-application/506-ros2-connector directory: numpy, pytest and pytest-asyncio.
Updates
checkovfrom 3.2.529 to 3.3.6Release notes
Sourced from checkov's releases.
Changelog
Sourced from checkov's changelog.
... (truncated)
Commits
4f04622Merge 295d3c344d020ebf7b97b63d810e1d25a68309b1 into 9c6a42eabe339592684d196ad...9c6a42efix(terraform): prevent crash in S3AllowsAnyPrincipal with unparsed v… (#7581)d32961achore: update release notes07a94feMerge a5dfacf9f3fb4d597825bfbbb9794ee44431fb96 into 0896e312b080f94b4be25fd45...b257af8fix(terraform): CKV_AZURE_190 override singular get_expected_value (#7592)0896e31feat(general): fix danger node version (#7589)f47b0cdfix(general): retry kustomize/helm install to survive transient GitHub failur...2e66fadfeat(general): fix danger node version (#7589)6b8a613feat(general): fix danger node version (#7589)63dad53chore: update release notesUpdates
numpyfrom 2.4.6 to 2.5.0Release notes
Sourced from numpy's releases.
... (truncated)
Changelog
Sourced from numpy's changelog.
... (truncated)
Commits
6910b28Merge pull request #31706 from charris/prepare-2.5.0-releasee0acd2bREL: Prepare for the NumPy 2.5.0 release.8d928b7Merge pull request #31704 from charris/backport-31649c2055baMAINT: update openblas to 0.3.33.112.0 (#31649)ce17c81Merge pull request #31703 from charris/backport-316093de6203BUG: fix StringDType distinct-allocator bugs and add tests (#31609)c723971Merge pull request #31700 from charris/backport-3169464513b2MAINT: Bump pypa/cibuildwheel from 3.4.1 to 4.1.004707f0Merge pull request #31698 from charris/try-fix-emscripten5cf0686MAINT: Try to fix emscripten wheel build.Updates
pytestfrom 9.0.3 to 9.1.1Release notes
Sourced from pytest's releases.
... (truncated)
Commits
cf470ecPrepare release version 9.1.1e0c8ce6Merge pull request #14625 from pytest-dev/patchback/backports/9.1.x/a07c31a97...1b82d16Merge pull request #14624 from pytest-dev/patchback/backports/9.1.x/b375b79ec...501c4bcMerge pull request #14596 from bluetech/doc-classmethodb61f588Merge pull request #14622 from chrisburr/fix-14608-initial-conftest-test-subdir9a567e0[automated] Update plugin list (#14617) (#14618)ef8b299Merge pull request #14620 from pytest-dev/patchback/backports/9.1.x/680f9f3ed...66abd07Merge pull request #14220 from bysiber/fix-stale-iexp-raisesgroup79fbf93Merge pull request #14612 from pytest-dev/patchback/backports/9.1.x/974ed48b6...0d312ebMerge pull request #14611 from bluetech/parametrize-argvalues-typingUpdates
pytest-asynciofrom 1.3.0 to 1.4.0Release notes
Sourced from pytest-asyncio's releases.
... (truncated)
Commits
6e14cd2chore: Prepare release of v1.4.0.4b900fbBuild(deps): Bump codecov/codecov-action from 6.0.0 to 6.0.1ab9f632Build(deps): Bump zipp from 3.23.1 to 4.1.0a56fc77Build(deps): Bump hypothesis from 6.152.6 to 6.152.8e8bae9bBuild(deps): Bump requests from 2.34.0 to 2.34.2fc43340Build(deps): Bump idna from 3.14 to 3.15762eaf5Build(deps): Bump jaraco-functools from 4.4.0 to 4.5.0b62e222Build(deps): Bump click from 8.3.3 to 8.4.09190447Build(deps): Bump pydantic from 2.13.3 to 2.13.482a393cci: Remove unnecessary debug output.