Page MenuHomePhabricator

[reports] Verify CSAT if present
ClosedPublic

Authored by bartek on Jun 17 2024, 3:49 AM.
Tags
None
Referenced Files
F2831589: D12448.diff
Sat, Sep 28, 12:32 AM
Unknown Object (File)
Sat, Sep 14, 10:57 AM
Unknown Object (File)
Tue, Sep 10, 7:29 AM
Unknown Object (File)
Tue, Sep 10, 3:01 AM
Unknown Object (File)
Mon, Sep 9, 7:35 PM
Unknown Object (File)
Mon, Sep 9, 3:17 AM
Unknown Object (File)
Mon, Sep 9, 1:57 AM
Unknown Object (File)
Thu, Sep 5, 1:33 AM
Subscribers

Details

Summary

CSAT for uploading reports is optional. If it's present, reporting user ID can be retrieved from it.
It's also used for Blob service requests (report upload). If it's not present, a service-to-service token is retrieved.

This diff handles the case, when requesting UserIdentity (CSAT) is present, but Identity verification failed.
In this case, we need service-to-service token, the same as for non-existing credential.

Depends on D12416

Test Plan

Tested locally and on staging. Uploaded large (> 400kB) report with valid and invalid credentials.
Both uploads succeeded. Looking at Blob service logs (level=trace), noted that:

  • the former request used UserIdentity credential,
  • the latter request used ServicesToken credential.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable