Repository navigation
chore(deps): bump the gomod group across 1 directory with 3 updates - #824
Closed
dependabot[bot] wants to merge 1 commit into
Closed
dependabot[bot] wants to merge 1 commit into
dependabot[bot] wants to merge 1 commit into
Conversation
Bumps the gomod group with 3 updates in the /blueprints/full-multi-node-cluster/tests directory: [github.com/microsoftgraph/msgraph-sdk-go](https://github.com/microsoftgraph/msgraph-sdk-go), [k8s.io/api](https://github.com/kubernetes/api) and [k8s.io/apimachinery](https://github.com/kubernetes/apimachinery). Updates `github.com/microsoftgraph/msgraph-sdk-go` from 1.102.0 to 1.103.0 - [Release notes](https://github.com/microsoftgraph/msgraph-sdk-go/releases) - [Changelog](https://github.com/microsoftgraph/msgraph-sdk-go/blob/main/CHANGELOG.md) - [Commits](microsoftgraph/msgraph-sdk-go@v1.102.0...v1.103.0) Updates `k8s.io/api` from 0.37.0 to 0.37.1 - [Commits](kubernetes/api@v0.37.0...v0.37.1) Updates `k8s.io/apimachinery` from 0.37.0 to 0.37.1 - [Commits](kubernetes/apimachinery@v0.37.0...v0.37.1) --- updated-dependencies: - dependency-name: github.com/microsoftgraph/msgraph-sdk-go dependency-version: 1.103.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: gomod - dependency-name: k8s.io/api dependency-version: 0.37.1 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: gomod - dependency-name: k8s.io/apimachinery dependency-version: 0.37.1 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: gomod ... Signed-off-by: dependabot[bot] <support@github.com>
9 of 29 tasks
Alain Uyidi (auyidi1)
added a commit
that referenced
this pull request
Oct 5, 2026
# Pull Request ## Description `main` currently fails the Grype step of the **Security Scan** job, which fails the required `PR Validation Gate` on unrelated PRs (for example #831, #834, #825, #824, #797, and #786). Running Grype 0.109.1 with `.grype.yaml` against `main` (`128cd7ae`) reports three findings at or above the `high` threshold: | Package | Version | Advisory | Fix | Location | |---------|---------|----------|-----|----------| | `basic-ftp` | 5.3.1 | GHSA-c475-qrg2-pj4r (published 2026-10-01) | 6.2.1 | `.github/scripts/markdown-link-check/package-lock.json` | | `braces` | 3.0.3 | GHSA-vfj7-8cjw-p6xm (published 2026-09-18) | None | `docs/docusaurus/package-lock.json` | | `http-cache-semantics` | 4.2.0 | GHSA-ch52-4w7c-c8xp (published 2026-09-18) | None | `docs/docusaurus/package-lock.json` | This PR remediates the fixable finding and documents the two unfixable ones. ## Related Issue Relates to #831 ## Type of Change - [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 ### `basic-ftp` override * Latest `get-uri` (8.0.1) still requires `basic-ftp ^5.3.1`, so there is no upstream upgrade path. * Added an npm `overrides` entry for `basic-ftp: ^6.2.2` in `.github/scripts/markdown-link-check/package.json` and updated the existing root `overrides` and `resolutions` from `^5.3.0` to `^6.2.2`. 6.2.2 also fixes GHSA-5rfr-xx34-2xxv. * `get-uri` only calls `access`, `lastMod`, `list`, `downloadTo`, and `close`. The only 6.0.0 breaking change disables separate transfer hosts by default to prevent FTP bounce attacks; none of these call sites depend on it. * Lockfile changes are limited to the `basic-ftp` entry in each lockfile. ### Scoped Grype ignores Both advisories have no patched release. The ignores follow the existing `image-size` precedent and are scoped by package type and name: * `braces` (GHSA-vfj7-8cjw-p6xm): reached through Docusaurus build-time file watching and glob matching (`chokidar`, `micromatch`) with repository-controlled patterns. * `http-cache-semantics` (GHSA-ch52-4w7c-c8xp): reached only through Docusaurus's update notifier (`update-notifier` → `latest-version` → `package-json` → `got` → `cacheable-request`), a single-user CLI version check. The repository does not run a shared HTTP cache. Both ignores should be removed when patched releases are available. ## Testing Performed - [ ] Terraform plan/apply - [ ] Blueprint deployment test - [ ] Unit tests - [ ] Integration tests - [ ] Bug fix includes regression test (see [Test Policy](docs/contributing/testing-validation.md)) - [x] Manual validation - [x] Other: Grype 0.109.1 with `.grype.yaml` (same version and config as CI) ## Validation Steps 1. `grype dir:. --config .grype.yaml` on `main`: exit 2 with the three high findings above. 2. Same command on this branch: exit 0 with no high or critical findings. 3. `yamllint -c .yamllint.yml .grype.yaml`: pass. 4. `npm ci --prefix .github/scripts/markdown-link-check` installs `basic-ftp` 6.2.2, and `markdown-link-check` runs and checks links normally. ## Checklist - [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 - [x] Lint checks pass (run applicable linters for changed file types) ## Security Review - [x] No credentials, secrets, or tokens are hardcoded or logged - [ ] RBAC and identity changes follow least-privilege principles - [ ] 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 No Terraform, Bicep, or application code changes. The two Grype ignores are risk acceptances and need maintainer review. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
|
Looks like these dependencies are updatable in another way, so this is no longer needed. |
dependabot
Bot
deleted the
dependabot/go_modules/blueprints/full-multi-node-cluster/tests/gomod-387ef258cb
branch
October 5, 2026 16:31
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 gomod group with 3 updates in the /blueprints/full-multi-node-cluster/tests directory: github.com/microsoftgraph/msgraph-sdk-go, k8s.io/api and k8s.io/apimachinery.
Updates
github.com/microsoftgraph/msgraph-sdk-gofrom 1.102.0 to 1.103.0Release notes
Sourced from github.com/microsoftgraph/msgraph-sdk-go's releases.
Changelog
Sourced from github.com/microsoftgraph/msgraph-sdk-go's changelog.
Commits
79ea66bMerge pull request #1034 from microsoftgraph/release-please--branches--main--...f4856c0chore(main): release 1.103.098c6092Merge pull request #1032 from microsoftgraph/v1.0/pipelinebuild/241182cde2ccdfeat(generation): update request builders and modelsUpdates
k8s.io/apifrom 0.37.0 to 0.37.1Commits
49d8ceaUpdate dependencies to v0.37.1 tagUpdates
k8s.io/apimachineryfrom 0.37.0 to 0.37.1Commits
e3a4b17Update dependencies to v0.37.1 tagDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore <dependency name> major versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)@dependabot ignore <dependency name> minor versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)@dependabot ignore <dependency name>will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)@dependabot unignore <dependency name>will remove all of the ignore conditions of the specified dependency@dependabot unignore <dependency name> <ignore condition>will remove the ignore condition of the specified dependency and ignore conditions