fix(ci): pass E2E_TOTP_SECRET secret into the E2E job env #107

Merged

nrupard merged 1 commit from fix/e2e-wire-totp-secret into main

2026-06-05 18:37:03 +02:00

nrupard commented

2026-06-05 18:36:27 +02:00

Owner

Summary

PR #106 added preflight + loginViaSpa support for E2E_TOTP_SECRET but missed wiring it through the workflow env block. Even with the Forgejo Actions secret set, the job never sees it and preflight fails with the now-correct "E2E_TOTP_SECRET missing" message.

One-line fix.

Test plan

Operator adds E2E_TOTP_SECRET Forgejo Actions secret if not already done.
Merge. Next E2E run clears preflight; setup either captures bearer (TOTP path works) or fails with the existing diagnostic dump for further iteration.

## Summary PR #106 added preflight + `loginViaSpa` support for `E2E_TOTP_SECRET` but missed wiring it through the workflow env block. Even with the Forgejo Actions secret set, the job never sees it and preflight fails with the now-correct "E2E_TOTP_SECRET missing" message. One-line fix. ## Test plan - [ ] Operator adds `E2E_TOTP_SECRET` Forgejo Actions secret if not already done. - [ ] Merge. Next E2E run clears preflight; setup either captures bearer (TOTP path works) or fails with the existing diagnostic dump for further iteration.

nrupard added 1 commit

2026-06-05 18:36:27 +02:00

fix(ci): pass E2E_TOTP_SECRET secret into the E2E job env

Create release / Create release from merged PR (pull_request) Has been skipped

Details

Check / fmt + clippy + compile + tests (pull_request) Successful in 1m55s

Details

5bcaf8d4cc

PR #106 added preflight + login-side support for the second factor but did not wire E2E_TOTP_SECRET through the workflow env block, so even if the operator adds the secret in Forgejo, the job never sees it and preflight fails with the now-accurate "E2E_TOTP_SECRET missing" message.

#PMS-140