Skip to content

fix(trust-graph): count discovery 5xx failures#100

Merged
EfeDurmaz16 merged 1 commit into
mainfrom
fides-discovery-circuit-5xx
May 6, 2026
Merged

fix(trust-graph): count discovery 5xx failures#100
EfeDurmaz16 merged 1 commit into
mainfrom
fides-discovery-circuit-5xx

Conversation

@EfeDurmaz16

Copy link
Copy Markdown
Owner

Summary

  • count discovery HTTP 5xx responses toward the identity-resolution circuit breaker
  • reuse the same failure path for transport errors and upstream 5xx responses
  • add regression coverage that the circuit opens after repeated 5xx responses and stops calling discovery

Verification

  • pnpm --filter @fides/trust-graph exec vitest run test/trust-service.test.ts -t "opens the discovery circuit breaker"
  • pnpm --filter @fides/trust-graph test
  • pnpm --filter @fides/trust-graph lint
  • pnpm build
  • git diff --check

@EfeDurmaz16 EfeDurmaz16 merged commit 83e4dd8 into main May 6, 2026
9 checks passed
@EfeDurmaz16 EfeDurmaz16 deleted the fides-discovery-circuit-5xx branch May 6, 2026 18:37
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