[identity] Treat empty auth metadata as invalid CSAT
ClosedPublic
Actions

Authored by bartek on Tue, Oct 1, 7:52 AM.

Details

Reviewers

varun
will

Commits

rCOMMe2f9c9a9db31: [identity] Treat empty auth metadata as invalid CSAT

Summary

ENG-9458.
Empty userID or deviceID makes DDB complain.

Test Plan

Called VerifyUserAccessToken RPC with empty user ID. The RPC errored before, now returns false.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bartek created this revision.Tue, Oct 1, 7:52 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptTue, Oct 1, 7:52 AM

bartek published this revision for review.Tue, Oct 1, 7:55 AM

Harbormaster completed remote builds in B31986: Diff 44757.Tue, Oct 1, 8:12 AM

varun accepted this revision.Tue, Oct 1, 10:26 AM

This revision is now accepted and ready to land.Tue, Oct 1, 10:26 AM

Closed by commit rCOMMe2f9c9a9db31: [identity] Treat empty auth metadata as invalid CSAT (authored by bartek). · Explain WhyTue, Oct 1, 11:07 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMMe2f9c9a9db31: [identity] Treat empty auth metadata as invalid CSAT.

Revision Contents
Changeset List

Path

Size

services/

identity/

src/

database/

token.rs

7 lines

Diff 44791

View Options

services/identity/src/database/token.rs

[identity] Treat empty auth metadata as invalid CSATClosedPublicActions