Page MenuHomePhabricator
Differential D12418

[lib] Add validator support for 'Unauthenticated' Tunnelbroker message
ClosedPublic
Actions

Authored by bartek on Jun 13 2024, 2:44 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 1, 10:31 AM
Unknown Object (File)
Fri, Nov 1, 10:31 AM
Unknown Object (File)
Fri, Nov 1, 10:31 AM
Unknown Object (File)
Fri, Nov 1, 10:31 AM
Unknown Object (File)
Sep 25 2024, 10:48 PM
Unknown Object (File)
Sep 25 2024, 10:48 PM
Unknown Object (File)
Sep 25 2024, 10:48 PM
Unknown Object (File)
Sep 25 2024, 10:47 PM
View All 43 Files
Subscribers
tomek

Details

Reviewers
kamil
ashoat
Commits
rCOMMcf79e85283ea: [lib] Add validator support for 'Unauthenticated' Tunnelbroker message
Summary

Addresses ENG-8394.
The Unauthenticated enum variant was missing in client-side validator.

Added a comment to keep that in sync.

Test Plan
  1. Hardcoded setUnauthorizedDeviceID(not-null-value) to keep Tunnelbroker connection anonymous.
  2. Called tunnelbrokerContext.sendMessage()
  3. Verified that invalid TunnelbrokerMessage is no longer occuring

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek created this revision.Jun 13 2024, 2:44 AM
bartek held this revision as a draft.
Herald added a subscriber: tomek. · View Herald TranscriptJun 13 2024, 2:44 AM
Harbormaster completed remote builds in B29650: Diff 41273.Jun 13 2024, 3:01 AM
bartek published this revision for review.Jun 13 2024, 10:02 PM
kamil accepted this revision.Jun 14 2024, 1:44 AM
kamil added inline comments.
shared/tunnelbroker_messages/src/messages/message_to_device_request_status.rs
12–13 ↗(On Diff #41273)

We already have a general comment one level higher in mod.rs, bur probably adding this here wouldn't hurt

This revision is now accepted and ready to land.Jun 14 2024, 1:44 AM
Closed by commit rCOMMcf79e85283ea: [lib] Add validator support for 'Unauthenticated' Tunnelbroker message (authored by bartek). · Explain WhyJun 14 2024, 2:43 AM
This revision was automatically updated to reflect the committed changes.
bartek added a commit: rCOMMcf79e85283ea: [lib] Add validator support for 'Unauthenticated' Tunnelbroker message.

Revision Contents
Changeset List

PathSize
lib/
types/
tunnelbroker/
4 lines
shared/
tunnelbroker_messages/
src/
messages/
4 lines

Diff 41318

View Options

lib/types/tunnelbroker/message-to-device-request-status-types.js

Loading...
View Options

shared/tunnelbroker_messages/src/messages/message_to_device_request_status.rs

Loading...