Paths

Table of Contentst

Differential D12895

[tunnelbroker][lib] Add WebPush notif message type
ClosedPublic
Actions

Authored by bartek on Jul 26 2024, 1:25 PM.

Tags

None

Referenced Files

None

Subscribers

Details

Reviewers

kamil
marcin

Commits

rCOMM61bfe5dd37da: [tunnelbroker][lib] Add WebPush notif message type

Summary

Message send from client to send notif using WebPush service.

Depends on D12894

Test Plan

Tested together with the rest of the stack.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bartek created this revision.Jul 26 2024, 1:25 PM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptJul 26 2024, 1:26 PM

bartek added a child revision: D12896: [tunnelbroker] Implement sending WebPush notifs by clients.Jul 26 2024, 1:27 PM

bartek published this revision for review.Jul 26 2024, 1:40 PM

Harbormaster failed remote builds in B30710: Diff 42833!Jul 26 2024, 1:49 PM

bartek mentioned this in D12901: [shared][rust] Add proc-macro to deserialize tagged enum variants.Jul 28 2024, 2:12 PM

Add TagAwareDeserialize to the struct. This should fix CI

bartek added a parent revision: D12903: [util-macros] Improve TagAwareDeserialize to read serde attributes.Jul 28 2024, 2:22 PM

Harbormaster completed remote builds in B30724: Diff 42850.Jul 28 2024, 2:36 PM

kamil accepted this revision.Jul 29 2024, 9:12 AM

kamil added inline comments.

shared/tunnelbroker_messages/src/messages/notif.rs
29 ↗	(On Diff #42850)	Is it worth adding `TagAwareDeserialize` to all structs used here? I think there are other having the same structure and only tag is different

This revision is now accepted and ready to land.Jul 29 2024, 9:12 AM

bartek added inline comments.Jul 29 2024, 12:52 PM

shared/tunnelbroker_messages/src/messages/notif.rs
29 ↗	(On Diff #42850)	Yes it is, but I also want to unit test all of them, Created https://linear.app/comm/issue/ENG-8905/add-tagawaredeserialize-to-messages-and-test-serialization to track this

Closed by commit rCOMM61bfe5dd37da: [tunnelbroker][lib] Add WebPush notif message type. · Explain WhyJul 29 2024, 12:56 PM

This revision was automatically updated to reflect the committed changes.

bartek mentioned this in rCOMM9463274996c6: [shared][rust] Add proc-macro to deserialize tagged enum variants.

bartek added a commit: rCOMM61bfe5dd37da: [tunnelbroker][lib] Add WebPush notif message type.

Revision Contents
Changeset List

Path

Size

lib/

types/

tunnelbroker/

12 lines

shared/

tunnelbroker_messages/

1 line

src/

messages/

1 line

12 lines

Diff 42890

Cargo.lock

Loading...

lib/types/tunnelbroker/notif-types.js

Loading...

shared/tunnelbroker_messages/Cargo.toml

Loading...

shared/tunnelbroker_messages/src/messages/mod.rs

Loading...

shared/tunnelbroker_messages/src/messages/notif.rs

Loading...