Paths

Table of Contentst

Differential D13990

[lib][keyserver] Handle invalid CSAT for multi-holder actions
ClosedPublic
Actions

Authored by bartek on Nov 21 2024, 11:59 AM.

Tags

None

Referenced Files

	F32816432: D13990.1768045595.diff
	Fri, Jan 9, 11:46 AM

Subscribers

Details

Reviewers

kamil
varun
ashoat

Commits

rCOMMf4e33a13ca08: [lib][keyserver] Handle invalid CSAT for multi-holder actions

Summary

Part of ENG-9526

Depends on D13989

Test Plan

Manual testing with mocked invalid access token

Diff Detail

Repository

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Nov 21 2024, 11:59 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptNov 21 2024, 11:59 AM

bartek added a child revision: D13991: [native][web] Handle invalid CSAT in fetchAndDecryptMedia.Nov 21 2024, 12:00 PM

Harbormaster completed remote builds in B32746: Diff 45921.Nov 21 2024, 12:23 PM

bartek published this revision for review.Nov 25 2024, 7:01 AM

bartek edited the test plan for this revision. (Show Details)Nov 25 2024, 7:24 AM

ashoat requested changes to this revision.Nov 25 2024, 1:49 PM

ashoat added inline comments.

lib/actions/holder-actions.js
58	Why is this optional?
109	Why is this optional?
152	Why are we not passing `invalidTokenLogOut` here?

This revision now requires changes to proceed.Nov 25 2024, 1:49 PM

bartek added inline comments.Nov 25 2024, 3:10 PM

lib/actions/holder-actions.js
58	To skip handling this error in holder cleanup step, which happens during logout
109	Same as above / below, it's just passed deeper into call sites
152	This is a holder cleanup, it's already called during logout. If the CSAT is invalid here, server-side cleanup will take care of this.

Thanks for explaining!

This revision is now accepted and ready to land.Nov 25 2024, 9:20 PM

Rebase

Harbormaster completed remote builds in B32852: Diff 46067.Nov 26 2024, 2:38 PM

Closed by commit rCOMMf4e33a13ca08: [lib][keyserver] Handle invalid CSAT for multi-holder actions. · Explain WhyNov 29 2024, 1:03 PM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMMf4e33a13ca08: [lib][keyserver] Handle invalid CSAT for multi-holder actions.

Revision Contents
Changeset List

Path

Size

keyserver/

src/

services/

5 lines

lib/

actions/

holder-actions.js

18 lines

handlers/

holders-handler.react.js

7 lines

push/

send-hooks.react.js

3 lines

utils/

blob-service.js

8 lines

Diff 45921

keyserver/src/services/blob.js

Loading...

lib/actions/holder-actions.js

Loading...

lib/handlers/holders-handler.react.js

Loading...

lib/push/send-hooks.react.js

Loading...

lib/utils/blob-service.js

Loading...