Repository navigation
build(deps): Bump the misc-dependencies group across 1 directory with 12 updates - #9831
dependabot[bot] wants to merge 1 commit into
Conversation
|
Pipeline controller notification For optional jobs, comment This repository is configured in: LGTM mode |
|
Important Review skippedWe couldn't safely recover the incremental review. No full review was started, and the last reviewed checkpoint was preserved. Retry later, or explicitly request a full review by commenting You can disable this status message by setting the Use the checkbox below for a quick retry:
Walkthrough
Priority: Merge Risk:
|
| Check name | Status | Explanation | Resolution |
|---|---|---|---|
| No-Sensitive-Data-In-Logs | ❌ Error | The dependency update adds a new Regional Access Boundary request log that can expose an access token. In `vendor/cloud.google.com/go/auth/internal/regionalaccessboundary/regional_access_boundary.go:1… | Do not log the raw authenticated request. Redact or omit Authorization, cookies, API keys, and other credential-bearing headers before constructing the log value, and avoid logging sensitive request or response payloads. Update or patch t… |
✅ Passed checks (10 passed)
-
Check name Status Explanation Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled. Title check ✅ Passed The title clearly describes the main change: a Dependabot update of 12 miscellaneous dependencies in one directory. It is concise and specific. Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request. Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request. Stable And Deterministic Test Names ✅ Passed The pull request changes dependency manifests, checksums, and vendored production sources. The authoritative diff contains no _test.gofiles and no added Ginkgo title calls (It,Describe, `Conte…Test Structure And Quality ✅ Passed PASS: The pull request changes dependency manifests, go.sum, and vendored dependency code only. The review-scoped diff contains no *_test.go files and no added Ginkgo constructs such as It, BeforeEach… Topology-Aware Scheduling Compatibility ✅ Passed The pull request changes only dependency metadata ( go.mod,go.sum) and vendored dependency files. It does not modify deployment manifests, operator code, or controllers. The diff introduces no top…Ipv6 And Disconnected Network Test Compatibility ✅ Passed The pull request changes only go.mod, go.sum, and vendored dependency files. It adds no Ginkgo e2e tests or other test files, so the IPv4 and external-connectivity conditions do not apply. No-Weak-Crypto ✅ Passed PASS. The PR updates Go dependencies and vendored code. The added code contains only crypto/tlsandcrypto/x509imports. No added MD5, SHA1, DES, RC4, 3DES, Blowfish, or ECB API usage was found. T…Container-Privileges ✅ Passed The pull request changes go.mod, go.sum, and vendored dependency files only. It does not change container or Kubernetes manifests. No added lines introduce privileged, hostPID, hostNetwork, hostIPC, S…
Full details: No-Sensitive-Data-In-Logs
-
Explanation
The dependency update adds a new Regional Access Boundary request log that can expose an access token. In
vendor/cloud.google.com/go/auth/internal/regionalaccessboundary/regional_access_boundary.go:124-132, the code putstyp + " " + token.Valueinto theAuthorizationheader and then logs the request withinternallog.HTTPRequest. The formatter invendor/github.com/googleapis/gax-go/v2/internallog/internallog.go:74-82serializes every request header without redaction. The path is wired into credential detection by the changeddetect.gocode and is reachable when Regional Access Boundary support is enabled. Logging is disabled by default, but a configured logger orGOOGLE_SDK_GO_LOGGING_LEVELcan enable it. This matches the check's token exposure condition.Resolution
Do not log the raw authenticated request. Redact or omit
Authorization, cookies, API keys, and other credential-bearing headers before constructing the log value, and avoid logging sensitive request or response payloads. Update or patch the vendored cloud-auth/gax logging implementation and add a regression test that verifies an access-token value never appears in debug logs.
✨ Finishing Touches: 💡 1
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
🛠️ Fix failing CI checks: 💡
-
- Commit to this branch
- Create a new PR
-
Comment @coderabbitai help to get the list of available commands.
|
Hi @dependabot[bot]. Thanks for your PR. I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with Regular contributors should join the org to skip this step. Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
234707d to
533110d
Compare
2a6dab1 to
6ac08b3
Compare
9fc6a97 to
8cff586
Compare
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
7288fae to
d249474
Compare
… 12 updates --- updated-dependencies: - dependency-name: go.etcd.io/etcd/api/v3 dependency-version: 3.7.2 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: misc-dependencies - dependency-name: go.etcd.io/etcd/client/pkg/v3 dependency-version: 3.7.2 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: misc-dependencies - dependency-name: go.etcd.io/etcd/client/v3 dependency-version: 3.7.2 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: misc-dependencies - dependency-name: go.etcd.io/etcd/server/v3 dependency-version: 3.7.2 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: misc-dependencies - dependency-name: go.opentelemetry.io/otel dependency-version: 1.46.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: misc-dependencies - dependency-name: go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc dependency-version: 1.46.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: misc-dependencies - dependency-name: go.opentelemetry.io/otel/sdk dependency-version: 1.46.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: misc-dependencies - dependency-name: go.opentelemetry.io/otel/trace dependency-version: 1.46.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: misc-dependencies - dependency-name: google.golang.org/api dependency-version: 0.299.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: misc-dependencies - dependency-name: google.golang.org/grpc dependency-version: 1.84.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: misc-dependencies - dependency-name: kubevirt.io/api dependency-version: 1.9.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: misc-dependencies - dependency-name: kubevirt.io/containerized-data-importer-api dependency-version: 1.66.1 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: misc-dependencies ... Signed-off-by: dependabot[bot] <support@github.com>
d249474 to
d5a3d8f
Compare
|
@dependabot[bot]: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Dependabot 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 conditionsSummary by CodeRabbit