fix: split Woodpecker Forgejo URL for OAuth vs clones #65
No reviewers
Labels
No labels
domain:backend
domain:devops
domain:frontend
status:approved
status:in-progress
status:needs-fix
status:qa
type:bug
type:devops
type:feature
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
forgejo_admin/pal-e-platform!65
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "hotfix-woodpecker-oauth-url"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Summary
WOODPECKER_FORGEJO_URLinto two: external URL for OAuth browser redirects, internal URL for clone operationsforgejo-http.forgejo.svc.cluster.localwhich browsers can't resolveChanges
terraform/main.tf:WOODPECKER_FORGEJO_URL→ external Tailscale URL (for OAuth), newWOODPECKER_FORGEJO_CLONE_URL→ internal service URL (for clones)Test Plan
tofu apply -target=helm_release.woodpecker— OAuth login worksReview Checklist
Related
plan-pal-e-platformReview — Pass
Files changed: 1 (
terraform/main.tf, +2 / -1)Findings
Minimal, correct hotfix. No issues.
WOODPECKER_FORGEJO_URL— Restored to external Tailscale URL usingvar.tailscale_domaininterpolation (matchesWOODPECKER_HOSTpattern). Required for browser OAuth redirects.WOODPECKER_FORGEJO_CLONE_URL— New env var with the internal service URL. Woodpecker uses this for clone operations instead ofFORGEJO_URLwhen set. This preserves the TLS fix from PR #56.Already live — Applied via local
tofu apply -target=helm_release.woodpecker. OAuth login verified via Playwright. This PR codifies the fix in main.No secrets, no unrelated changes, alignment preserved.
Tofu Plan Output