Page MenuHomePhabricator
Differential D10804

[shared] [identity] [2/n] Add identity_search_messages to shared and use in identity
ClosedPublic
Actions

Authored by will on Jan 24 2024, 1:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jul 5, 7:00 PM
Unknown Object (File)
Mon, Jul 1, 11:38 AM
Unknown Object (File)
Mon, Jul 1, 11:38 AM
Unknown Object (File)
Mon, Jul 1, 11:38 AM
Unknown Object (File)
Mon, Jul 1, 11:38 AM
Unknown Object (File)
Mon, Jul 1, 11:38 AM
Unknown Object (File)
Mon, Jul 1, 11:38 AM
Unknown Object (File)
Mon, Jul 1, 11:37 AM
View All 53 Files
Subscribers
ashoat
tomek

Details

Reviewers
varun
bartek
Commits
rCOMMe4cdbf3fee9c: [shared] [identity] [2/n] Add identity_search_messages to shared and use in…
Summary

This introduces identity_search_messages into shared for usage by identity's prefix search websocket services and commtest (in later diffs).

Depends on D10802

Test Plan

cargo fmt and build. Successfully running identity service

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

will created this revision.Jan 24 2024, 1:24 PM
will held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptJan 24 2024, 1:24 PM
will published this revision for review.Jan 24 2024, 1:25 PM
Harbormaster completed remote builds in B26128: Diff 36101.Jan 24 2024, 1:43 PM
will updated this revision to Diff 36103.Jan 24 2024, 2:20 PM

make connection initialization messages pub

Harbormaster completed remote builds in B26129: Diff 36103.Jan 24 2024, 2:40 PM
will planned changes to this revision.Jan 26 2024, 7:44 AM
will requested review of this revision.Jan 26 2024, 2:48 PM
will retitled this revision from Add identity_search_messages to shared and use in identity to [services] [2/n] Add identity_search_messages to shared and use in identity.Jan 26 2024, 2:51 PM
will added a child revision: D10847: [identity] [3/n] move websocket response functions to send.rs.
varun accepted this revision.Jan 29 2024, 7:35 PM
varun added inline comments.
shared/identity_search_messages/src/messages/auth_messages.rs
1–2 ↗(On Diff #36103)

This doc comment seemingly doesn't match the actual contents of the file. Guessing there's more to come here but would be good to have the comment accurately reflect what's in here

This revision is now accepted and ready to land.Jan 29 2024, 7:35 PM
will updated this revision to Diff 36345.Jan 29 2024, 9:07 PM

Update doc comments

Harbormaster completed remote builds in B26298: Diff 36345.Jan 29 2024, 9:36 PM
will retitled this revision from [services] [2/n] Add identity_search_messages to shared and use in identity to [shared] [identity] [2/n] Add identity_search_messages to shared and use in identity.Jan 30 2024, 11:06 AM
bartek accepted this revision.Jan 31 2024, 1:58 AM
will updated this revision to Diff 36430.Jan 31 2024, 10:41 AM

Add ConnectionInitializationResponse to Messages

Harbormaster completed remote builds in B26359: Diff 36430.Jan 31 2024, 11:08 AM
Closed by commit rCOMMe4cdbf3fee9c: [shared] [identity] [2/n] Add identity_search_messages to shared and use in… (authored by will). · Explain WhyJan 31 2024, 11:10 AM
This revision was automatically updated to reflect the committed changes.
will added a commit: rCOMMe4cdbf3fee9c: [shared] [identity] [2/n] Add identity_search_messages to shared and use in….

Revision Contents
Changeset List

PathSize
services/
identity/
1 line
src/
websockets/
12 lines
shared/
identity_search_messages/
1 line
11 lines
src/
3 lines
messages/
13 lines
18 lines

Diff 36436

View Options

services/identity/Cargo.lock

Loading...
View Options

services/identity/Cargo.toml

Loading...
View Options

services/identity/src/websockets/auth.rs

Loading...
View Options

shared/identity_search_messages/.gitignore

Loading...
View Options

shared/identity_search_messages/Cargo.lock

Loading...
View Options

shared/identity_search_messages/Cargo.toml

Loading...
View Options

shared/identity_search_messages/src/lib.rs

Loading...
View Options

shared/identity_search_messages/src/messages/auth_messages.rs

Loading...
View Options

shared/identity_search_messages/src/messages/mod.rs

Loading...