[identity] Delete unauthenticated UploadOneTimeKeys RPC
ClosedPublic
Actions

Authored by bartek on Mon, Nov 20, 4:15 AM.

Details

Reviewers

varun

Commits

rCOMMa4da1e841133: [identity] Delete unauthenticated UploadOneTimeKeys RPC

Summary

We have UploadOneTimeKeys in both authenticated and unauthenticated proto. I've removed the unauthenticated one and changed the usages to use the authenticated one.

I was curious why this wasn't done earlier, see: https://phab.comm.dev/D9059#267156

Test Plan

Ran Commtest tests changing its endpoint to use the authenticated one.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bartek created this revision.Mon, Nov 20, 4:15 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptMon, Nov 20, 4:15 AM

bartek added a child revision: D9929: [identity] Extract insert_into_cache to trait.Mon, Nov 20, 4:17 AM

Harbormaster completed remote builds in B24268: Diff 33406.Mon, Nov 20, 4:36 AM

bartek published this revision for review.Mon, Nov 20, 4:50 AM

just one question inline

services/commtest/tests/identity_access_tokens_tests.rs
38 ↗	(On Diff #33406)	should we modify this test to use the auth client instead of deleting it altogether?

This revision is now accepted and ready to land.Mon, Nov 20, 11:35 AM

bartek added inline comments.Mon, Dec 4, 2:39 AM

services/commtest/tests/identity_access_tokens_tests.rs
38 ↗	(On Diff #33406)	I deleted it because it was duplicated. Look at `upload_one_time_keys()` test one file above. I think this was Jon's rebase issue when he was introducing it.