Skip to content

fix(streaming): stop reporting retryable ClosedSendChannelException to Bugsnag#978

Merged
bmc08gt merged 1 commit into
code/cashfrom
fix/suppress-bidi-initial-send-bugsnag-noise
Jun 24, 2026
Merged

fix(streaming): stop reporting retryable ClosedSendChannelException to Bugsnag#978
bmc08gt merged 1 commit into
code/cashfrom
fix/suppress-bidi-initial-send-bugsnag-noise

Conversation

@bmc08gt

@bmc08gt bmc08gt commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

The initial-request send in openBidirectionalStream already treats ClosedSendChannelException as retryable, but the trace() call was passing error = e which routed through ErrorUtils → Bugsnag.notify(). Inline the message instead so the retry path no longer produces false-positive Bugsnag warnings.

…o Bugsnag

The initial-request send in openBidirectionalStream already treats
ClosedSendChannelException as retryable, but the trace() call was
passing `error = e` which routed through ErrorUtils → Bugsnag.notify().
Inline the message instead so the retry path no longer produces
false-positive Bugsnag warnings.

Signed-off-by: Brandon McAnsh <[email protected]>
@bmc08gt bmc08gt self-assigned this Jun 24, 2026
@github-actions github-actions Bot added area: network gRPC, connectivity, API, exchange rates type: fix Bug fix labels Jun 24, 2026
@bmc08gt bmc08gt merged commit d985eb8 into code/cash Jun 24, 2026
3 checks passed
@bmc08gt bmc08gt deleted the fix/suppress-bidi-initial-send-bugsnag-noise branch June 24, 2026 23:08
bmc08gt added a commit that referenced this pull request Jun 24, 2026
…inator-delegates

* origin/code/cash:
  feat(messaging): integrate updated chat protos with data layer and tests (#976)
  fix(streaming): stop reporting retryable ClosedSendChannelException to Bugsnag (#978)
  build: update release manifest

# Conflicts:
#	apps/flipcash/shared/chat/src/main/kotlin/com/flipcash/shared/chat/ChatCoordinator.kt
#	apps/flipcash/shared/chat/src/test/kotlin/com/flipcash/shared/chat/ChatCoordinatorEventsTest.kt
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: network gRPC, connectivity, API, exchange rates type: fix Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant