fix: WOODPECKER_HOST in docker-compose.yml overrides .env — OAuth2 redirect still mismatches #178
Labels
No labels
action
backlog
blocked
bug-report
in-progress
prediction/actioned
prediction/dismissed
prediction/unreviewed
priority
tech-debt
underspecified
vision
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: disinto-admin/disinto#178
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
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?
Problem
The fix in #172 added
WOODPECKER_HOST=http://localhost:8000to.env, but docker-compose.yml hardcodes the value:Docker Compose
environment:entries override.envvalues, so the fix has no effect. Woodpecker still sendshttp://woodpecker:8000/authorizeas its OAuth2 redirect URI, while Forgejo hashttp://localhost:8000/authorizeregistered. Result:Fix
In the
generate_compose()function inbin/disinto, change the hardcodedWOODPECKER_HOSTto reference the.envvariable:This way the
.envvalue fromcreate_woodpecker_oauth()takes effect, while keeping a sensible default for standalone usage.Files
bin/disinto—generate_compose()function, the woodpecker service environment block