HomePhabricator
Diffusion Comm c459cea60ff1

[reports] Require auth for viewing reports

Description

[reports] Require auth for viewing reports

Summary:
Wrapped viewing reports endpoints with auth middleware.
The POST /reports still works without auth - it is optional there and used only for retrieving user_id.

Depends on D12415

Test Plan:

  • Repeated test plan from D12414 for /reports endpoints.
  • Verified that sending reports still works without auth.
  • Verified that user_id is present when sending reports with valid credentials.

Reviewers: will, varun, kamil

Reviewed By: varun

Subscribers: ashoat, tomek

Differential Revision: https://phab.comm.dev/D12416

Details

Provenance
bartekAuthored on Jun 13 2024, 12:40 AM
Reviewer
varun
Differential Revision
D12416: [reports] Require auth for viewing reports
Parents
rCOMM496f450b731f: [blob] Hide endpoints behind auth middleware
Branches
Unknown
Tags
Unknown