Skip to content

CI: replace invalid placeholder APP_SHARED_SECRET in Run Tests & Build Check - #5

Closed
DanielJohn17 with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-run-tests-build-check
Closed

DanielJohn17 with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-run-tests-build-check

Conversation

Copilot AI commented Sep 29, 2026 •

Copy link
Copy Markdown

The Run Tests & Build Check workflow failed because CI used the development placeholder APP_SHARED_SECRET, which is rejected by production-mode config validation at test startup. This change updates CI to use a non-placeholder secret value consistently across test and build steps.

  • Root cause alignment

    • Replaced dev-insecure-client-shared-secret-min-32-chars-long in workflow env with a CI-safe non-placeholder secret so config bootstrap no longer exits in production mode.
  • Workflow consistency

    • Updated both Run Tests and Build Test API & TUI steps to use the same APP_SHARED_SECRET.
    • Updated TUI -ldflags default app secret injection to match the same CI value.
- name: Run Tests
  env:
    APP_SHARED_SECRET: ci-test-shared-secret-not-for-production-usage-32
  run: go test -v ./...

- name: Build Test API & TUI
  env:
    APP_SHARED_SECRET: ci-test-shared-secret-not-for-production-usage-32
  run: |
    go build -v ./cmd/api
    go build -ldflags "-X main.DefaultAppSecret=ci-test-shared-secret-not-for-production-usage-32" -v ./cmd/tui

Co-authored-by: DanielJohn17 <112425917+DanielJohn17@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job Run Tests & Build Check CI: replace invalid placeholder APP_SHARED_SECRET in Run Tests & Build Check Sep 29, 2026
Copilot AI requested a review from DanielJohn17 September 29, 2026 22:19
@DanielJohn17

Copy link
Copy Markdown
Owner

Done locally

@DanielJohn17
DanielJohn17 deleted the copilot/fix-run-tests-build-check branch September 29, 2026 22:34
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.

2 participants