Skip to content

SCAL-321502-200-fix - #34

Merged
mnk-blr merged 2 commits into
mainfrom
SCAL-321502-200-fix
Jul 16, 2026
Merged

SCAL-321502-200-fix#34
mnk-blr merged 2 commits into
mainfrom
SCAL-321502-200-fix

Conversation

@yinstardev

@yinstardev yinstardev commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

200 - error paths covered , previous pr covered non-200 error paths - but graphql has 200-hapy error paths.

Scenario Pre-fix Now Verified
Happy path (admin) 200 200 + data live ✅
Privilege denied (eureka → UPSTREAM_FAILURE) 500 {} 403 live ✅
Bad/expired token (401 HTML downstream) 500/leak 401, clean body live ✅
Downstream validation error (200+errors) 500 {} 400 live ✅
503 down, numeric-403, app-code→500 clamp 500/undefined correct harness 12/12 ✅
Unit tests 44/44 ✅
Internal-URL/debug leakage in bodies leaked gone live ✅
Spec/SDK impact none verified

@codesandbox

codesandbox Bot commented Jul 15, 2026

Copy link
Copy Markdown

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

Comment thread test/index.ts
Comment thread src/common.ts Outdated
@mnk-blr
mnk-blr merged commit 292bccd into main Jul 16, 2026
3 checks passed
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