Promote to Production: CAR-1276 Phase 1 — auth /health 503 error-log fix #286

Merged

Coupon Carl merged 3 commits from uat into main 2026-06-06 00:25:10 +00:00

Conversation 0 Commits 3 Files Changed 2

+35 -6

Coupon Carl commented 2026-06-06 00:25:03 +00:00

Owner

Copy Link

Copy Source

CAR-1276 Phase 1 — auth /health 503 error-log fix

Promotes commit f283d5aa from uat to main.

Changes

• auth/src/index.ts: empty catch now console.errors DB-probe failure (name + message); 503 JSON body gains an error field carrying the message.
• No happy-path behaviour change: /health 200 contract unchanged.

Sign-offs

• UAT PASS — Deal Dottie (CAR-1282)
• Security PASS — Stockboy Steve (CAR-1282)

CI Notes

• lint, test, e2e, audit, build-and-push-auth all PASS
• build-and-push (frontend) and deploy-uat failures are pre-existing infra issues (tracked CAR-1213/CAR-1216/CAR-1218), unrelated to this auth-only change.

cc @cpfarhood

## CAR-1276 Phase 1 — auth /health 503 error-log fix Promotes commit `f283d5aa` from `uat` to `main`. ### Changes - `auth/src/index.ts`: empty catch now console.errors DB-probe failure (name + message); 503 JSON body gains an `error` field carrying the message. - No happy-path behaviour change: /health 200 contract unchanged. ### Sign-offs - **UAT PASS** — Deal Dottie (CAR-1282) - **Security PASS** — Stockboy Steve (CAR-1282) ### CI Notes - lint, test, e2e, audit, build-and-push-auth all PASS - build-and-push (frontend) and deploy-uat failures are pre-existing infra issues (tracked CAR-1213/CAR-1216/CAR-1218), unrelated to this auth-only change. cc @cpfarhood

Coupon Carl added 3 commits 2026-06-06 00:25:04 +00:00

fix(auth): log /health 503 error and surface message in body (CAR-1276) ... b2c4692400

The /health handler's catch block was empty, so when the DB probe
failed we had no log line to diagnose from. UAT auth was crashlooping
on /health 503s for that exact reason — pod logs only showed
'CartSnitch auth service listening on port 3001' and nothing else.

Add console.error with the error name/message and include the message
in the 503 response body so the next time this fails we can read the
actual error from `kubectl logs` without re-deploying.

This is the dev-side observability half of CAR-1276. The underlying
DB failure still needs investigation (likely better-auth schema
missing from the cartsnitch DB; see CAR-1276 for the analysis).

Tests updated to assert the new error field is present and a string.
Co-Authored-By: Paperclip <noreply@paperclip.ing>

fix(auth): log /health 503 error and surface message in body (#283, CAR-1276) 39804135a4

promote: auth /health 503 error-log fix (CAR-1276 Phase 1) dev→uat (#285) f283d5aa02

Coupon Carl merged commit c27f6a1e3c into main 2026-06-06 00:25:10 +00:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

bug

Something isn't working

documentation

Improvements or additions to documentation

duplicate

This issue or pull request already exists

enhancement

New feature or request

good first issue

Good for newcomers

help wanted

Extra attention is needed

invalid

This doesn't seem right

marketing

Marketing content

question

Further information is requested

wontfix

This will not be worked on

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

ai-review (AI Review) cs_betty (Barcode Betty) cs_charlie (Checkout Charlie) cpfarhood (Chris Farhood) ci (Continuous Integration [bot]) cs_carl (Coupon Carl) cs_dottie (Deal Dottie) flux (Flux CD) admin (Gitea Admin) cs_martha (Markdown Martha) renovate (Mend Renovate) cs_savannah (Savannah Savings) cs_steve (Stockboy Steve)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: cartsnitch/cartsnitch#286