[tunnelbroker] implement handling Farcaster requests
ClosedPublic
Actions

Authored by kamil on Aug 11 2025, 10:22 AM.

Details

Reviewers

bartek
tomek

Commits

rCOMM95af435f8d02: [tunnelbroker] implement handling Farcaster requests

Summary

ENG-10869.

Depends on D15186

Test Plan

Tested using some local tests (with mocked DC token and user ID), it was also tested with client changes in the actual app, example here.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Aug 11 2025, 10:22 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptAug 11 2025, 10:22 AM

kamil added a child revision: D15188: [lib] implement sending Farcaster requests.Aug 11 2025, 10:28 AM

kamil published this revision for review.Aug 11 2025, 10:48 AM

bartek accepted this revision.Aug 11 2025, 1:08 PM

bartek added inline comments.

services/tunnelbroker/src/websockets/session.rs
266–268 ↗	(On Diff #49498)	This makes the flow more obvious and easier to follow. You can name `message_from_device` differently though.

This revision is now accepted and ready to land.Aug 11 2025, 1:08 PM

Harbormaster failed remote builds in B35247: Diff 49498!Aug 11 2025, 6:40 PM

Harbormaster completed remote builds in B35247: Diff 49498.Aug 12 2025, 2:10 AM

address review

Harbormaster completed remote builds in B35276: Diff 49544.Aug 13 2025, 12:05 PM

Closed by commit rCOMM95af435f8d02: [tunnelbroker] implement handling Farcaster requests. · Explain WhyAug 13 2025, 1:22 PM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM95af435f8d02: [tunnelbroker] implement handling Farcaster requests.

Revision Contents
Changeset List

Path

Size

services/

tunnelbroker/

src/

websockets/

session.rs

78 lines

Diff 49548

View Options

services/tunnelbroker/src/websockets/session.rs

[tunnelbroker] implement handling Farcaster requestsClosedPublicActions