Paths

Table of Contentst

Differential D11549

[protos] Remove social_proof from IdentityKeyInfo
ClosedPublic
Actions

Authored by bartek on Apr 4 2024, 12:44 PM.

Tags

None

Referenced Files

None

Subscribers

Details

Reviewers

ashoat
varun

Commits

rCOMMe7f9f1ef90fb: [protos] Remove social_proof from IdentityKeyInfo

Summary

Addresses ENG-7530.

Removed the already-unused social_proof field from IdentityKeyInfo proto message type.

Depends on D11548

Test Plan

Everything still compiles, commtest able to call login/registration and X3DH RPCs as before.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bartek created this revision.Apr 4 2024, 12:44 PM

bartek held this revision as a draft.

Herald added a subscriber: tomek. · View Herald TranscriptApr 4 2024, 12:44 PM

Harbormaster failed remote builds in B27971: Diff 38768!Apr 4 2024, 12:58 PM

One more leftover

Harbormaster completed remote builds in B27973: Diff 38770.Apr 4 2024, 1:28 PM

bartek published this revision for review.Apr 4 2024, 1:30 PM

bartek added inline comments.

services/identity/src/database.rs
350–359 ↗	(On Diff #38768)	The comment was out of date There's no need to get full user data by it's `user_id` just to retrieve `user_id` 😉

bartek added a child revision: D11551: [protos] Replace challengeResponse with nonce and signature pair.Apr 4 2024, 1:59 PM

varun added inline comments.Apr 4 2024, 6:40 PM

services/identity/src/database.rs
350–359 ↗	(On Diff #38768)	haha yeah i was really confused by this code... thanks for cleaning it up!

amazing, thank you!

ashoat accepted this revision.Apr 5 2024, 1:45 PM

ashoat added inline comments.

web/grpc/identity-service-client-wrapper.js
201 ↗	(On Diff #38770)	Why do we use `toObject` here but not in `getKeyserverKeys`? Is it because we need to iterate here, but not in `getKeyserverKeys`?

This revision is now accepted and ready to land.Apr 5 2024, 1:45 PM

bartek added a subscriber: kamil.Apr 12 2024, 10:12 AM

bartek added inline comments.

web/grpc/identity-service-client-wrapper.js
201 ↗	(On Diff #38770)	Not sure, but probably yes. Accessing `response.getDevicesMap()` returns `Map<string, OutboundKeyInfoObject>`, but `toObject().devicesMap` returns an array of tuples. cc @kamil who authored this code

Closed by commit rCOMMe7f9f1ef90fb: [protos] Remove social_proof from IdentityKeyInfo. · Explain WhyApr 15 2024, 8:44 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMMe7f9f1ef90fb: [protos] Remove social_proof from IdentityKeyInfo.

ashoat mentioned this in D11664: [native] Remove socialProof from outbound keyserver keys blobs.Apr 15 2024, 1:12 PM

ashoat mentioned this in rCOMM02d8d4de3f42: [native] Remove socialProof from outbound keyserver keys blobs.Apr 15 2024, 1:45 PM

Revision Contents
Changeset List

Path

Size

keyserver/

addons/

rust-node-addon/

src/

identity_client/

get_inbound_keys_for_user.rs

1 line

1 line

4 lines

register_user.rs

1 line

lib/

types/

identity-service-types.js

4 lines

native/

native_rust_library/

src/

identity/

3 lines

registration.rs

2 lines

6 lines

services/

commtest/

src/

identity/

2 lines

identity/

src/

15 lines

database/

3 lines

shared/

protos/

identity_unauth.proto

3 lines

web/

grpc/

identity-service-client-wrapper.js

2 lines

protobufs/

identity-unauth-structs.cjs

identity-unauth-structs.cjs.flow

6 lines

Diff 39104

keyserver/addons/rust-node-addon/src/identity_client/get_inbound_keys_for_user.rs

Loading...

keyserver/addons/rust-node-addon/src/identity_client/login.rs

Loading...

keyserver/addons/rust-node-addon/src/identity_client/mod.rs

Loading...

keyserver/addons/rust-node-addon/src/identity_client/register_user.rs

Loading...

lib/types/identity-service-types.js

Loading...

native/native_rust_library/src/identity/login.rs

Loading...

native/native_rust_library/src/identity/registration.rs

Loading...

native/native_rust_library/src/identity/x3dh.rs

Loading...

services/commtest/src/identity/device.rs

Loading...

services/identity/src/database.rs

Loading...

services/identity/src/database/device_list.rs

Loading...

shared/protos/identity_unauth.proto

Loading...

web/grpc/identity-service-client-wrapper.js

Loading...

web/protobufs/identity-unauth-structs.cjs

Loading...

web/protobufs/identity-unauth-structs.cjs.flow

Loading...