Skip to content

fix(gitea-runner): status token fallback and env-prefix policy#917

Closed
AlexMikhalev wants to merge 7 commits into
mainfrom
task/runner-status-fallback-env-policy
Closed

fix(gitea-runner): status token fallback and env-prefix policy#917
AlexMikhalev wants to merge 7 commits into
mainfrom
task/runner-status-fallback-env-policy

Conversation

@AlexMikhalev

Copy link
Copy Markdown
Contributor

Summary

Fixes two native runner infra issues blocking green native-ci / build (push) on private repos:

  1. HTTP 401 on commit status — per-job github.token can checkout but lacks /statuses scope. Runner now prefers RUNNER_STATUS_TOKEN / GITEA_TOKEN for native commit-status posts (job token remains fallback).
  2. Policy reject RUSTDOC=$(rustup — Gitea inlines step env: into run: commands. Policy now strips leading shell assignments (including $(subshell) forms) before allowlist checks.

Deploy

Already deployed on bigbox: binary rebuilt from this branch, RUNNER_STATUS_TOKEN added to all three runner env files, services restarted.

Test plan

  • cargo test -p terraphim_gitea_runner (33 tests)
  • workflow_dispatch on terraphim-agents — verify no 401 / policy reject in runner logs

Refs terraphim/terraphim-ai#2464 (Gitea)

Test User added 7 commits June 13, 2026 10:53
…2596

Step 0 architecture claims verified. Action 1 implemented on
terraphim-agents task/2596-adf-issue-dedup (PR #43). Actions 2/4
pending bigbox SSH; Action 3 applied to 7 repos; Action 5 blocked
by open issue dependencies on first close attempt.
Prefer RUNNER_STATUS_TOKEN/GITEA_TOKEN for native commit-status posts when
the per-job github.token returns HTTP 401 on private repos. Strip leading
shell VAR=value assignments (including $(subshell) forms) before allowlist
checks so inlined workflow env does not reject cargo/rustup steps.

Refs #2464
@AlexMikhalev

Copy link
Copy Markdown
Contributor Author

Merged via Gitea PR #2665 (ec81404). GitHub main synced in 14b90b1.

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.

1 participant