Page MenuHomePhabricator

[reports] Verify CSAT if present
ClosedPublic

Authored by bartek on Jun 17 2024, 3:49 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 21, 2:55 PM
Unknown Object (File)
Sat, Dec 21, 2:55 PM
Unknown Object (File)
Sat, Dec 21, 2:55 PM
Unknown Object (File)
Sat, Dec 21, 2:54 PM
Unknown Object (File)
Sun, Dec 15, 3:41 PM
Unknown Object (File)
Tue, Dec 3, 11:12 AM
Unknown Object (File)
Fri, Nov 29, 7:03 AM
Unknown Object (File)
Wed, Nov 27, 9:07 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