decentraland / social-service-ea / 28526473436

90%

main: 90%

Build Type

Pull #424

github

Committed by LautaroPetaccio

Commit Message

fix: harden the subscription path (non-fatal snapshot, no-emitter backoff, drop metric)

Server-side robustness for the subscription streams, complementing the client-side
re-subscribe fixes:

- subscribe-to-friend-connectivity: make the initial online-friends snapshot best-effort.
  A DB/registry hiccup no longer tears down the whole subscription (which forced the client
  to reconnect and re-run the queries, churning); log a warning and continue to live updates.
  (peersStats.getConnectedPeers already swallows its own errors, so the real risk was
  getOnlineFriends / getProfiles.)
- handleSubscriptionUpdates: on the "no emitter" path (RPC call live but the per-address
  emitter is gone — an abnormal/racy state), throw instead of returning. A clean completion
  invited an instant re-open into a hot loop; erroring makes a well-behaved client back off.
- emitterToGenerator: add an onOverflowDrop hook + subscription_updates_dropped_total metric
  so silent value-queue drops (a slow consumer missing updates) are observable.
- Keep the per-occurrence "Duplicate subscription detected" debug log alongside the
  subscription_duplicates_total metric, for per-connection visibility into re-subscribe churn.

Build, lint and full unit suite (106 suites / 1615 tests) green.

Pull Request Pull Request #424: fix: harden the subscription path (non-fatal snapshot, no-emitter backoff, drop metric)

Coverage Stats

2296 of 2685 branches covered (85.51%)

Branch coverage included in aggregate %.

5 of 6 new or added lines in 3 files covered. (83.33%)

2 existing lines in 1 file now uncovered.

5756 of 6221 relevant lines covered (92.53%)

80.77 hits per line