Skip to content

🐛 deps: bump golang.org/x/net to clear govulncheck finding - #27

Closed
clubanderson wants to merge 1 commit into
mainfrom
fix/govulncheck-xnet
Closed

clubanderson wants to merge 1 commit into
mainfrom
fix/govulncheck-xnet

Conversation

@clubanderson

Copy link
Copy Markdown
Member

Summary

govulncheck (known vulnerabilities) is red on main, holding every PR with base-branch-red. All four findings are in golang.org/x/net@v0.59.0:

  • GO-2026-6617
  • GO-2026-6612
  • GO-2026-6611
  • GO-2026-6603

Bump golang.org/x/net v0.59.0 → v0.62.0 (pulls golang.org/x/sys v0.48.0 → v0.49.0) and go mod tidy. Same fix hive applied in hive#14731 / #14753.

Verification

  • go build ./... clean
  • govulncheck ./... → No vulnerabilities found.

Fixes #26

Signed-off-by: Andy Anderson <andy@clubanderson.com>
@kubestellar-prow

Copy link
Copy Markdown

[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.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hivecommons-hive

Copy link
Copy Markdown
Contributor

@clubanderson is auto-promoted for contributor-lane merging on this PR: the last 5 merged PRs by this author in this hive's repositories have no revert or sentinel-alert evidence. CI, reviewer verdict, diff fences, soak, holds and branch protection still apply.

@hivecommons-hive hivecommons-hive Bot added author/human A person pushed this PR themselves dco All commits are DCO signed-off role/trusted Author is trusted for unattended merge (trusted-author role or registry/claim tier in require_tiers) stage/050-review-pending Review swarm requested; awaiting a verdict for the current head labels Oct 28, 2026
@hivecommons-hive

Copy link
Copy Markdown
Contributor
029daf6

ℹ️ Sentinel notice — informational; author is trusted (owner), merge is not blocked

Hive flagged this PR (author @clubanderson, head 029daf6bb75d) because it matches behaviors that can override security controls, escalate privileges or damage the codebase. This is a heuristic, not an accusation — a maintainer should confirm the change is intended before it merges.

  • sensitive_path — changes 1 sensitive path(s) (Touches a sensitive path (OWNERS, workflows, policies, hive config, security docs, …))
    • go.mod

Hive did not add the sentinel-alert label because this author is trusted. The finding remains recorded for audit and the dashboard Security tab; set sentinel.trusted_authors_block: true to require the blocking label for trusted authors too.

@hivecommons-hive hivecommons-hive Bot added stage/071-needs-author Human-authored PR: CI red, changes requested or conflicting; diagnosis posted, the author must act and removed stage/050-review-pending Review swarm requested; awaiting a verdict for the current head labels Oct 28, 2026
@clubanderson

Copy link
Copy Markdown
Member Author

Closing: crux's guard check refuses Go edits here because src/crux in hivecommons/hive is the source of truth. The bump is open as hivecommons/hive#14782 (base v5) and will be published here by the crux sync once it merges. Tracking stays on #26.

@clubanderson
clubanderson deleted the fix/govulncheck-xnet branch October 28, 2026 01:18
@hivecommons-hive

Copy link
Copy Markdown
Contributor

@clubanderson this PR is held at stage/071-needs-author and the hive will not merge it yet.

Failing checks:

  • check-failure

Next: the hive does not push to your branch. Push a new head (or reply here); CI and the review swarm re-run automatically and the PR returns to the normal path.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author/human A person pushed this PR themselves dco All commits are DCO signed-off dco-signoff: yes role/trusted Author is trusted for unattended merge (trusted-author role or registry/claim tier in require_tiers) size/S stage/071-needs-author Human-authored PR: CI red, changes requested or conflicting; diagnosis posted, the author must act

Projects

None yet

Development

Successfully merging this pull request may close these issues.

govulncheck red on main: GO-2026-6617, GO-2026-6612, GO-2026-6611, GO-2026-6603 (golang.org/x/net)

1 participant