[backup][backup-client] Add auth to logs websocket
Needs ReviewPublic
Actions

Authored by bartek on Thu, Jun 20, 4:56 AM.

Details

Reviewers

marcin
varun
will
kamil

Summary

Addresses 3 in ENG-8473.

Added LogWSResponse::AuthSuccess
Modified backup service handler to verify passed UserIdentity
Modified backup client to wait for AuthSuccess message after sending auth request.

Depends on D12508

Test Plan

Backup integration tests. Added some logging.
Also tested with COMM_SERVICES_SKIP_CSAT_VERIFICATION - auth message is still required but verification always succeeds.

Diff Detail

Repository

rCOMM Comm

Branch

barthap/dns

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Thu, Jun 20, 4:56 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptThu, Jun 20, 4:56 AM

Harbormaster completed remote builds in B29829: Diff 41565.Thu, Jun 20, 5:25 AM

bartek published this revision for review.Thu, Jun 20, 7:19 AM

bartek added inline comments.

services/backup/src/http/handlers/log.rs
61	Unfortunately we need this to modify this `Option` from spawned future and read it in the sync fn. Technically, they're not going block by each other but we need to do it for safety.