Page MenuHomePhabricator
Differential D11546

[protos][identity] Make keyserver_info response non-optional
ClosedPublic
Actions

Authored by bartek on Apr 3 2024, 8:20 AM.
Tags
None
Referenced Files
Unknown Object (File)
Apr 16 2024, 4:32 PM
Unknown Object (File)
Apr 15 2024, 8:36 PM
Unknown Object (File)
Apr 11 2024, 11:46 AM
Unknown Object (File)
Apr 11 2024, 11:46 AM
Unknown Object (File)
Apr 10 2024, 7:29 PM
Unknown Object (File)
Apr 10 2024, 5:14 PM
Unknown Object (File)
Apr 10 2024, 11:29 AM
Unknown Object (File)
Apr 9 2024, 12:30 PM
View All 11 Files
Subscribers
tomek

Details

Reviewers
ashoat
varun
Commits
rCOMMd8727804c52e: [protos][identity] Make keyserver_info response non-optional
Summary

Resolves ENG-7598.
We agreed that keyserver_info shouldn't be optional.

Checked native_rust_library code and web identity client. They both already treat this as required so nothing to do client-side

Depends on D11471

Test Plan
  • Called the GetKeyserverKeys RPC for user without a keyserver - returned "keyserver not found"
  • Registered a keyserver
  • Now the RPC responded as before

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
services/
identity/
src/
grpc_services/
8 lines
shared/
protos/
2 lines
web/
protobufs/
2 lines

Diff 38887

View Options

services/identity/src/grpc_services/authenticated.rs

Loading...
View Options

shared/protos/identity_auth.proto

Loading...
View Options

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

Loading...