chore(deps): bump github.com/hashicorp/go-getter from 1.8.3 to 1.8.6#271
chore(deps): bump github.com/hashicorp/go-getter from 1.8.3 to 1.8.6#271dependabot[bot] wants to merge 1 commit intomasterfrom
Conversation
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 0 |
| Duplication | 0 |
AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.
TIP This summary will be updated as you push new changes. Give us feedback
There was a problem hiding this comment.
Pull Request Overview
Although Codacy analysis indicates the PR is up to standards, it should not be merged in its current state due to security vulnerabilities identified in the requested Go toolchain (1.25.8). Specifically, this version is susceptible to XSS (CVE-2026-32289) and DoS (CVE-2026-32281, CVE-2026-32288) attacks. Furthermore, there is a significant gap between the PR title and the actual changes, which include substantial updates to the Go toolchain and various cloud SDKs.
About this PR
- The update involves a high number of indirect dependencies across critical SDKs. This increases the risk surface beyond a standard maintenance bump; verify that no breaking changes are introduced by these secondary updates.
- The scope of this PR is much broader than the title suggests. It includes a Go toolchain update and updates to several major library suites (AWS, Google Cloud SDK, and OpenTelemetry). Please update the PR metadata to reflect the full scope of these changes to ensure they are properly audited.
Test suggestions
- Verify successful compilation and basic functionality of resource fetching using the upgraded
go-getterlibrary. - Verify that the project build environment supports the non-standard Go version 1.25.8 specified in
go.mod.
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Verify successful compilation and basic functionality of resource fetching using the upgraded `go-getter` library.
2. Verify that the project build environment supports the non-standard Go version 1.25.8 specified in `go.mod`.
🗒️ Improve review quality by adding custom instructions
| module github.com/codacy/codacy-trivy | ||
|
|
||
| go 1.25.6 | ||
| go 1.25.8 |
There was a problem hiding this comment.
🔴 HIGH RISK
The requested Go toolchain version 1.25.8 has multiple known vulnerabilities: XSS via improper context tracking in JS template literals (CVE-2026-32289), and Denial of Service (DoS) risks via certificate chain validation (CVE-2026-32281) and crafted archives (CVE-2026-32288). Additionally, version 1.25.8 appears to be a non-standard release for the current Go ecosystem. Upgrading to 1.25.9 is required to mitigate these security risks.
| go 1.25.8 | |
| go 1.25.9 |
Bumps [github.com/hashicorp/go-getter](https://github.com/hashicorp/go-getter) from 1.8.3 to 1.8.6. - [Release notes](https://github.com/hashicorp/go-getter/releases) - [Commits](hashicorp/go-getter@v1.8.3...v1.8.6) --- updated-dependencies: - dependency-name: github.com/hashicorp/go-getter dependency-version: 1.8.6 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
52735c0 to
4aacd19
Compare
Bumps github.com/hashicorp/go-getter from 1.8.3 to 1.8.6.
Release notes
Sourced from github.com/hashicorp/go-getter's releases.
... (truncated)
Commits
d23bff4Merge pull request #608 from hashicorp/dependabot/go_modules/go-security-9c51...2c4aba8Merge pull request #613 from hashicorp/pull/v1.8.6fe61ed9Merge pull request #611 from hashicorp/SECVULN-41053d533656Merge pull request #606 from hashicorp/pull/CRT388f23dAdditional test for local branch and headb7ceaa5harden checkout ref handling and added regression tests769cc14Release version bump up6086a6aReview Comments Addressede02063cRevert "SECVULN Fix for git checkout argument injection enables arbitrary fil...c93084d[chore] : Bump google.golang.org/grpc