Skip to content

fix: evaluate the commit pinned by the Go module proxy in private mode - #164

Merged
traefiker merged 2 commits into
traefik:masterfrom
mmatur:fix/evaluate-pinned-commit
Oct 29, 2026
Merged

traefiker merged 2 commits into
traefik:masterfrom
mmatur:fix/evaluate-pinned-commit

Conversation

@mmatur

@mmatur mmatur commented Oct 28, 2026 •

Copy link
Copy Markdown
Member

In private mode, piceus reads the plugin at the tag. plugin-service serves the commit the Go module proxy pinned at publication (traefik/plugin-service#83), so once a tag moved, piceus evaluates code that users never get.

piceus now reads the plugin (code, .traefik.yml, readme and go.mod) at the pinned commit, and at the tag only when the proxy pinned none, as plugin-service does. A version whose pinned code does not build is reported to its author like any other evaluation error.

Private repositories are not sent to the proxy. If the proxy still cannot be read after 3 attempts, the plugin is skipped for this run and no issue is opened.

@mmatur mmatur added kind/enhancement a new or improved feature. status/2-needs-review labels Oct 28, 2026

@darkweaver87 darkweaver87 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.

LGTM

Comment thread pkg/sources/github.go Resolved
Comment thread cmd/run/run.go Resolved
Comment thread pkg/sources/github.go Resolved
Comment thread pkg/core/yaegi.go Resolved
Comment thread cmd/run/run.go Resolved
Comment thread pkg/sources/github.go Resolved
The pinned commit is now found once, by the scrapper, with retries. A private repository is never sent to the Go module proxy, and a proxy failure has its own error.
@traefiker
traefiker merged commit bed9818 into traefik:master Oct 29, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/enhancement a new or improved feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants