Paths

Table of Contentst

[lib][keyserver] Handle invalid CSAT in BlobOperationResult
ClosedPublic
Actions

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

Details

Reviewers

kamil
varun
ashoat

Commits

rCOMM39f3157f8ad7: [lib][keyserver] Handle invalid CSAT in BlobOperationResult

Summary

Part of ENG-9526.

Depends on D13957, D13960

Test Plan

Manual testing with mocked invalid access token

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

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

bartek held this revision as a draft.

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

bartek added a child revision: D13989: [lib][keyserver] Handle invalid CSAT on blob download.Nov 21 2024, 11:59 AM

Harbormaster failed remote builds in B32744: Diff 45919!Nov 21 2024, 12:13 PM

Fix eslint

Harbormaster completed remote builds in B32750: Diff 45925.Nov 21 2024, 12:56 PM

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

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

ashoat accepted this revision.Nov 25 2024, 1:33 PM

ashoat added inline comments.

lib/utils/services-utils.js
38	Can we name this something more specific? Either `blobResponseIsInvalidCSAT` or `httpResponseIsInvalidCSAT` would be good, I think

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

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

lib/utils/services-utils.js
38	Yes, I'll do that. `httpResponseIsInvalidCSAT` sounds good

Rebase. Rename function to httpResponseIsInvalidCSAT

Harbormaster completed remote builds in B32840: Diff 46055.Nov 26 2024, 12:48 PM

Closed by commit rCOMM39f3157f8ad7: [lib][keyserver] Handle invalid CSAT in BlobOperationResult. · Explain WhyNov 29 2024, 1:03 PM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMM39f3157f8ad7: [lib][keyserver] Handle invalid CSAT in BlobOperationResult.

Revision Contents
Changeset List

Path

Size

keyserver/

src/

services/

blob.js

22 lines

lib/

actions/

upload-actions.js

3 lines

utils/

blob-service.js

28 lines

services-utils.js

6 lines

Diff 45925

View Options

keyserver/src/services/blob.js

View Options

lib/actions/upload-actions.js

View Options

lib/utils/blob-service.js

View Options

lib/utils/services-utils.js

[lib][keyserver] Handle invalid CSAT in BlobOperationResultClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 45925

keyserver/src/services/blob.js

lib/actions/upload-actions.js

lib/utils/blob-service.js

lib/utils/services-utils.js

[lib][keyserver] Handle invalid CSAT in BlobOperationResult
ClosedPublic
Actions

Revision Contents
Changeset List