Skip to content

Test/gitleaks#1518

Open
Ming1309 wants to merge 31 commits intonashtech-garage:mainfrom
Ming1309:test/gitleaks
Open

Test/gitleaks#1518
Ming1309 wants to merge 31 commits intonashtech-garage:mainfrom
Ming1309:test/gitleaks

Conversation

@Ming1309
Copy link
Copy Markdown

@Ming1309 Ming1309 commented May 1, 2026

No description provided.

Ming1309 and others added 30 commits April 21, 2026 22:16
Replace every hard-coded ghcr.io/nashtech-garage/ prefix with
ghcr.io/${{ github.repository_owner }}/ so that forks publish images
to their own GHCR namespace instead of the upstream org.

Also add explicit job-level permissions (contents: read, packages: write)
to every affected workflow so that the GITHUB_TOKEN is granted the
packages scope regardless of repository default settings.

Affected workflows (20 files, 22 references):
- backoffice-ci.yaml        (3 refs: build tag, image-ref, push tag)
- backoffice-bff-ci.yaml    (1 ref)
- storefront-ci.yaml        (1 ref)
- storefront-bff-ci.yaml    (1 ref)
- sampledata-ci.yaml        (1 ref)
- media-ci.yaml             (1 ref)
- cart-ci.yaml              (1 ref)
- customer-ci.yaml          (1 ref)
- inventory-ci.yaml         (1 ref)
- location-ci.yaml          (1 ref)
- order-ci.yaml             (1 ref)
- payment-ci.yaml           (1 ref)
- payment-paypal-ci.yaml    (1 ref)
- product-ci.yaml           (1 ref)
- promotion-ci.yaml         (1 ref)
- rating-ci.yaml            (1 ref)
- recommendation-ci.yaml    (1 ref)
- search-ci.yaml            (1 ref)
- tax-ci.yaml               (1 ref)
- webhook-ci.yaml           (1 ref)

No trigger or branch gate changes; publish-on-main behaviour is preserved.

Fixes: media-ci.yaml:91 GHCR push permission/namespace error
ci: make GHCR image namespace fork-safe via github.repository_owner
ci(backoffice): add checks: write and security-events: write permissions
ci: update gitleaks workflow configuration
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants