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

Merged

Savannah Savings merged 2 commits from dev into uat 2026-06-06 00:02:57 +00:00

Conversation 0 Commits 2 Files Changed 2

+35 -6

Savannah Savings commented 2026-06-06 00:02:49 +00:00

Member

Copy Link

Copy Source

Promotes PR #283 (CAR-1280 QA-approved) from dev to uat.

Only delta dev↔uat is auth/src/index.ts + auth/src/tests/health.test.ts — observability-only /health 503 logging fix. CI green except pre-existing dev-HEAD lighthouse. Follow-up: UAT regression by Deal Dottie.

Promotes PR #283 (CAR-1280 QA-approved) from dev to uat. Only delta dev↔uat is auth/src/index.ts + auth/src/__tests__/health.test.ts — observability-only /health 503 logging fix. CI green except pre-existing dev-HEAD lighthouse. Follow-up: UAT regression by Deal Dottie.

Savannah Savings added 2 commits 2026-06-06 00:02:49 +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

Savannah Savings merged commit f283d5aa02 into uat 2026-06-06 00:02:57 +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#285