Skip to content

feat: surface subscribeToStream polling errors - #426

Open
BrightHeart23 wants to merge 1 commit into
conduit-protocol:mainfrom
BrightHeart23:main
Open

feat: surface subscribeToStream polling errors#426
BrightHeart23 wants to merge 1 commit into
conduit-protocol:mainfrom
BrightHeart23:main

Conversation

@BrightHeart23

Copy link
Copy Markdown

Summary

  • invoke the optional StreamEventHandlers.onError callback when event polling fails
  • normalize non-Error rejection values while retaining the existing console warning
  • keep polling if the consumer's error handler throws

Validation

  • npm test -- --run src/tests/events-subscribe.test.ts (11 tests passed)
  • npm run typecheck
  • npm run lint -- --no-fix

Closes #200

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.

Enhancement: add an onError handler for subscribeToStream polling failures

1 participant