Skip to content

Comments

Fix deploy smoke path handling and public invoker access#19

Merged
gepser merged 1 commit intomasterfrom
feature/fix-smoke-path-default
Feb 22, 2026
Merged

Fix deploy smoke path handling and public invoker access#19
gepser merged 1 commit intomasterfrom
feature/fix-smoke-path-default

Conversation

@gepser
Copy link
Owner

@gepser gepser commented Feb 22, 2026

Summary

  • fix scripts/smoke.sh default-path behavior so PROGRESS_PATH="" stays empty (for direct Cloud Functions URLs)
  • deploy Cloud Functions Gen2 with allow_unauthenticated: true
  • keep smoke checks using deploy output URL without duplicating /progress

Why

The deploy job was failing for two reasons:

  1. smoke URL could become /progress/progress/:id
  2. after switching to Gen2, endpoint required auth and smoke got 403

Validation

  • mise exec -- make check
  • ✅ local smoke script tested with both forms:
    • BASE_URL=http://localhost:8080 (default path)
    • BASE_URL=http://localhost:8080/progress PROGRESS_PATH=""

Notes

  • This keeps the endpoint public like the existing markdown badge usage expects.
  • After merge, rerun the Deploy workflow on master.

@gepser gepser merged commit 5b5e135 into master Feb 22, 2026
2 checks passed
@gepser gepser deleted the feature/fix-smoke-path-default branch February 22, 2026 22:52
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.

1 participant