[lib] Add optional Authorization header to reports endpoint
ClosedPublic
Actions

Authored by bartek on Feb 2 2024, 1:10 AM.

Details

Reviewers

tomek
michal

Commits

rCOMMe9f3781d057f: [lib] Add optional Authorization header to reports endpoint

Summary

Part of ENG-6289.

Added a way to pass optional AuthMetadata to reports endpoint.
This header is optional because this endpoint should also work without any authentication.
Authentication gives us possibility to identify the user who is sending the report.

Depends on D10913

Test Plan

Tested together with the next diff.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bartek created this revision.Feb 2 2024, 1:10 AM

bartek held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptFeb 2 2024, 1:11 AM

bartek added a child revision: D10931: [lib][native] Pass CSAT to Reports endpoint.Feb 2 2024, 1:13 AM

Harbormaster completed remote builds in B26460: Diff 36578.Feb 2 2024, 1:39 AM

bartek published this revision for review.Feb 2 2024, 2:33 AM

tomek accepted this revision.Feb 6 2024, 1:40 AM

This revision is now accepted and ready to land.Feb 6 2024, 1:40 AM

Closed by commit rCOMMe9f3781d057f: [lib] Add optional Authorization header to reports endpoint (authored by bartek). · Explain WhyFeb 8 2024, 7:57 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMMe9f3781d057f: [lib] Add optional Authorization header to reports endpoint.

Revision Contents
Changeset List

Path

Size

lib/

actions/

report-actions.js

6 lines

utils/

reports-service.js

8 lines

Diff 36855

View Options

lib/actions/report-actions.js