Skip to content

fix(realtime): send heartbeat for initial connection error#1746

Merged
mandarini merged 1 commit into
masterfrom
fix/connError-heartbeat
Jan 22, 2026
Merged

fix(realtime): send heartbeat for initial connection error#1746
mandarini merged 1 commit into
masterfrom
fix/connError-heartbeat

Conversation

@mandarini

Copy link
Copy Markdown
Contributor

Moved from: supabase/realtime-js#507
Author: @bhavyasaggi

What kind of change does this PR introduce?

Add missing heartbeat on connection error.

What is the current behavior?

If the Supabase instance is unreachable during initial realtime connection, there is no status update available.

What is the new behavior?

Trigger a heartbeat-callback with 'error' on connection-error.

@mandarini mandarini requested review from a team as code owners October 8, 2025 13:37
@mandarini mandarini self-assigned this Oct 8, 2025
@mandarini mandarini added the realtime-js Related to the realtime-js library. label Oct 8, 2025
@coveralls

coveralls commented Oct 8, 2025

Copy link
Copy Markdown

Coverage Status

coverage: 80.931% (+0.02%) from 80.916%
when pulling 7fb75c7 on fix/connError-heartbeat
into 188b83f on master.

@mandarini mandarini force-pushed the fix/connError-heartbeat branch from d33afe6 to e2db0f0 Compare January 22, 2026 10:34
filipecabaco
filipecabaco previously approved these changes Jan 22, 2026
Comment thread packages/core/realtime-js/src/RealtimeClient.ts Outdated
@mandarini mandarini merged commit 16271de into master Jan 22, 2026
40 of 41 checks passed
@mandarini mandarini deleted the fix/connError-heartbeat branch January 22, 2026 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

realtime-js Related to the realtime-js library.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants