Paths

Table of Contentst

Differential D15177

[tunnelbroker] move `get_message_to_device_status` implementation to `MessageSentStatus`
ClosedPublic
Actions

Authored by kamil on Aug 8 2025, 8:31 AM.

Details

Reviewers

bartek

Commits

rCOMM006872ec5314: [tunnelbroker] move `get_message_to_device_status` implementation to…

Summary

ENG-10869.

Later in the stack, this is going to be used in more places than just WebsocketSession

Depends on D15176

Test Plan

commtest

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Aug 8 2025, 8:31 AM

kamil held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptAug 8 2025, 8:31 AM

kamil added a child revision: D15178: [tunnelbroker] refactor AMPQ code from `WebsocketSession` to a separate module.Aug 8 2025, 8:43 AM

kamil published this revision for review.Aug 8 2025, 12:01 PM

Harbormaster failed remote builds in B35061: Diff 49312!Aug 9 2025, 2:34 AM

Nice!

This revision is now accepted and ready to land.Aug 11 2025, 9:43 AM

This revision was landed with ongoing or failed builds.Aug 11 2025, 11:38 AM

Closed by commit rCOMM006872ec5314: [tunnelbroker] move `get_message_to_device_status` implementation to…. · Explain Why

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM006872ec5314: [tunnelbroker] move `get_message_to_device_status` implementation to….

Revision Contents
Changeset List

Path

Size

services/

tunnelbroker/

src/

websockets/

session.rs

90 lines

shared/

tunnelbroker_messages/

src/

messages/

message_to_device_request_status.rs

15 lines

Diff 49503

View Options

services/tunnelbroker/src/websockets/session.rs