forgejo_admin/pal-e-docs-app
1
0
Fork 0
Code Issues 5 Pull requests 2 Projects Releases Packages Wiki Activity Actions

fix: add auth header to wget for private pal-e-platform repo #89

Open
forgejo_admin wants to merge 1 commit from 226-fix-wget-auth into main
pull from: 226-fix-wget-auth
merge into: forgejo_admin:main
Conversation 0 Commits 1 Files changed 1 +1
forgejo_admin commented 2026-03-28 16:32:00 +00:00
Owner
Copy link

Summary

The update-kustomize-tag CI step downloads update-kustomize-tag.sh from pal-e-platform via wget, but gets a 404 because the repo is private. This adds the Authorization: token header using the existing FORGEJO_TOKEN secret.

Changes

  • .woodpecker.yaml: added --header="Authorization: token ${FORGEJO_TOKEN}" to the wget command in the update-kustomize-tag step

Test Plan

  • Merge to main and verify the update-kustomize-tag step succeeds in the next Woodpecker pipeline run (wget returns 200, script executes)
  • No regressions in other CI steps

Review Checklist

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

Related Notes

## Summary The `update-kustomize-tag` CI step downloads `update-kustomize-tag.sh` from pal-e-platform via wget, but gets a 404 because the repo is private. This adds the `Authorization: token` header using the existing `FORGEJO_TOKEN` secret. ## Changes - `.woodpecker.yaml`: added `--header="Authorization: token ${FORGEJO_TOKEN}"` to the wget command in the `update-kustomize-tag` step ## Test Plan - [ ] Merge to main and verify the `update-kustomize-tag` step succeeds in the next Woodpecker pipeline run (wget returns 200, script executes) - [ ] No regressions in other CI steps ## Review Checklist - [ ] Passed automated review-fix loop - [ ] No secrets committed - [ ] No unnecessary file changes - [ ] Commit messages are descriptive ## Related Notes - Closes forgejo_admin/pal-e-platform#226 - `pal-e-app` -- the project this work belongs to
The update-kustomize-tag.sh download was getting 404 because
pal-e-platform is a private repo. Add Authorization token header
using the existing FORGEJO_TOKEN secret.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This pull request has changes conflicting with the target branch.
  • .woodpecker.yaml
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin 226-fix-wget-auth:226-fix-wget-auth
git switch 226-fix-wget-auth

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff 226-fix-wget-auth
git switch 226-fix-wget-auth
git rebase main
git switch main
git merge --ff-only 226-fix-wget-auth
git switch 226-fix-wget-auth
git rebase main
git switch main
git merge --no-ff 226-fix-wget-auth
git switch main
git merge --squash 226-fix-wget-auth
git switch main
git merge --ff-only 226-fix-wget-auth
git switch main
git merge 226-fix-wget-auth
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
domain:backend

   
domain:devops

   
domain:frontend
No labels
domain:backend
domain:devops
domain:frontend
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
forgejo_admin/pal-e-docs-app!89
No description provided.