Skip to content

feat(slack): classify broker refresh failures - #240

Draft
mohammedahmed18 wants to merge 2 commits into
mainfrom
codex/slack-broker-errors
Draft

feat(slack): classify broker refresh failures#240
mohammedahmed18 wants to merge 2 commits into
mainfrom
codex/slack-broker-errors

Conversation

@mohammedahmed18

Copy link
Copy Markdown
Collaborator

Summary

  • Preserves sanitized permanent Slack refresh failures through the OAuth broker.
  • Decodes nested OAuth-service error envelopes in locality-slack while retaining legacy flat-envelope compatibility.
  • Adds service and client coverage for invalid-grant, authorization/scope, retryable upstream failures, and redaction.

Dependency

Required by the internal Slack broker worker PR so revoked/rotated credentials are classified as reconnect-required rather than retried indefinitely.

Verification

  • npm test and npm run typecheck in apps/oauth-service
  • cargo test -p locality-slack --locked (100 passed)
  • cargo fmt --check and git diff --check

@vercel

vercel Bot commented Aug 29, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
locality Ready Ready Preview Aug 29, 2026 2:31pm

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