8000 socat: add repology_project (un-blind the auto-update, #441) by bryan-minimal · Pull Request #357 · gominimal/pkgs · GitHub
[go: up one dir, main page]

Skip to content

socat: add repology_project (un-blind the auto-update, #441) - #357

Merged
twitchyliquid64 merged 2 commits into
mainfrom
socat-repology-provenance
Jul 6, 2026
Merged

twitchyliquid64 merged 2 commits into
mainfrom
socat-repology-provenance

Conversation

@bryan-minimal
@bryan-minimal bryan-minimal commented Jul 6, 2026 •
Copy link
Copy Markdown
Member

What

Adds repology_project = "socat" so pkgmgr's version-checker can discover socat's upstream releases.

Why

The hidden-update audit found socat is the lone confirmed hidden security update: pinned at 1.8.1.1, but stable upstream is 1.8.1.3 and 1.8.1.2 already fixes CVE-2026-56123 (HIGH — SOCKS5 DOMAINNAME heap overflow).

socat's origin (dest-unreach.org) is HTTP-only and fits no source_provenance category (not GitHub/GNU/SF/GitLab), so the version-checker had no tier to resolve and silently reported it up-to-date. Repology tracks socat and reports newest = 1.8.1.3, so repology_project is the right resolver.

This is the auto-update-blind flavor of the #441 provenance gap — socat isn't vuln-scan-dark (it matches CVE-2026-56123), but the missing provenance cost us the fix.

Two-change fix (this is the pkgs half)

  • This PR — repology_project for version discovery.
  • gominimal/pkgmgr-rs#449 — NAME_KEYED_UPSTREAMS entry so the gs:// mirror re-fetches the real tarball from dest-unreach.org on bump.

Once both land + deploy, the auto-update system cuts the socat → 1.8.1.3 PR itself (clears CVE-2026-56123).

Note

This PR only adds the resolver attr; it does not bump the version (socat stays 1.8.1.1). The bump comes from the auto-update system after this + #449 deploy. No source rebuild change.

Refs gominimal/pkgmgr-rs#441.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated the package’s build metadata to improve automated version tracking.
    • Version discovery now leverages Repology for this package, ensuring newer releases are detected more reliably.
    • Added explanatory notes about how version changes interact with the mirror and HTTPS integrity handling.

socat's upstream (dest-unreach.org) is HTTP-only and fits no
source_provenance category, so the version-checker resolved nothing and
reported socat up-to-date while 1.8.1.1 sat behind stable 1.8.1.3
(CVE-2026-56123 HIGH, fixed in 1.8.1.2). Add `repology_project = "socat"`
— repology reports newest = 1.8.1.3 — so the auto-update can discover
the bump. The gs:// mirror re-fetch on bump is handled by pkgmgr's
NAME_KEYED_UPSTREAMS (gominimal/pkgmgr-rs#449).

Surfaced by the hidden-update audit: socat was the lone confirmed hidden
security update among the directly-vulnerable packages.

Refs gominimal/pkgmgr-rs#441.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@coderabbitai
coderabbitai Bot commented Jul 6, 2026 •
Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: aeb9fce6-e5bf-4fe8-a0d9-399ea0e1d2a5

📥 Commits

Reviewing files that changed from the base of the PR and between 0330c5b and 310081a.

📒 Files selected for processing (1)
  • packages/socat/build.ncl
🚧 Files skipped from review as they are similar to previous changes (1)
  • packages/socat/build.ncl

📝 Walkthrough

Walkthrough

The socat package build metadata adds a repology_project attribute set to "socat" in attrs, with inline comments explaining repology-based upstream version discovery and HTTPS integrity behavior for the gs:// mirror.

Changes

Socat Build Metadata

Layer / File(s) Summary
Add repology_project attribute
packages/socat/build.ncl
Adds repology_project = "socat" to attrs with comments about repology version discovery and mirror integrity handling.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: msample

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: adding repology_project for socat to enable auto-update version discovery.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch socat-repology-provenance

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@packages/socat/build.ncl`:
- Around line 50-55: The inline comment in the socat build metadata has a PR
reference mismatch and should be corrected to point to the actual companion
change. Update the reference in the comment near repology_project so it matches
the NAME_KEYED_UPSTREAMS change cited in the PR description, and ensure the
gominimal/pkgmgr-rs PR number is consistent throughout the note.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d6f9d9d8-e2b5-4f3a-a825-2b1f4639af5f

📥 Commits

Reviewing files that changed from the base of the PR and between 2274e5e and 0330c5b.

📒 Files selected for processing (1)
  • packages/socat/build.ncl

Comment thread packages/socat/build.ncl Resolved
…(CR)

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@twitchyliquid64
twitchyliquid64 added this pull request to the merge queue Jul 6, 2026
Merged via the queue into main with commit d69d673 Jul 6, 2026
6 checks passed
@twitchyliquid64
twitchyliquid64 deleted the socat-repology-provenance branch July 6, 2026 22:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

0