fix: use YAML block scalar for wget auth header #134

Merged

forgejo_admin merged 1 commit from 226-fix-yaml-syntax into main

2026-03-28 16:58:44 +00:00

forgejo_admin commented

2026-03-28 16:57:38 +00:00

Owner

Summary

PR #128 added an --header="Authorization: token ..." argument to the wget command in the update-kustomize-tag step. The unquoted colon in that string caused YAML to interpret it as a mapping separator, breaking pipeline parsing entirely.

Changes

.woodpecker.yaml: Wrapped the multi-line wget command in a >- YAML block scalar so the colon inside the Authorization header is treated as plain string content, not a YAML mapping separator.

Test Plan

python3 -c "import yaml; yaml.safe_load(open('.woodpecker.yaml')); print('OK')" passes
Push to main triggers successful Woodpecker pipeline parse
No regressions in build-and-push or validate steps

Review Checklist

Passed automated review-fix loop
No secrets committed
No unnecessary file changes
Commit messages are descriptive

Closes #226
forgejo_admin/westside-app #128 -- the PR that introduced the broken syntax
westside-app -- the project this work belongs to

## Summary PR #128 added an `--header="Authorization: token ..."` argument to the wget command in the update-kustomize-tag step. The unquoted colon in that string caused YAML to interpret it as a mapping separator, breaking pipeline parsing entirely. ## Changes - `.woodpecker.yaml`: Wrapped the multi-line wget command in a `>-` YAML block scalar so the colon inside the Authorization header is treated as plain string content, not a YAML mapping separator. ## Test Plan - [x] `python3 -c "import yaml; yaml.safe_load(open('.woodpecker.yaml')); print('OK')"` passes - [ ] Push to main triggers successful Woodpecker pipeline parse - [ ] No regressions in build-and-push or validate steps ## Review Checklist - [x] Passed automated review-fix loop - [x] No secrets committed - [x] No unnecessary file changes - [x] Commit messages are descriptive ## Related Notes - Closes #226 - `forgejo_admin/westside-app #128` -- the PR that introduced the broken syntax - `westside-app` -- the project this work belongs to

forgejo_admin added 1 commit

2026-03-28 16:57:38 +00:00

fix: use YAML block scalar for wget command to fix broken parsing

ci/woodpecker/pr/woodpecker Pipeline was successful

Details

2dcba002ed

The colon in --header="Authorization: token ..." was interpreted as a
YAML mapping separator, breaking pipeline parsing. Wrapping the command
in a >- block scalar fixes this.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>