[reports-service] Add service error types
ClosedPublic
Actions

Authored by bartek on Aug 28 2023, 2:38 AM.

Tags

None

Referenced Files

	F2109660: D8968.id30426.diff
	Tue, Jun 25, 5:14 PM

	F2104099: D8968.id30510.diff
	Tue, Jun 25, 4:21 AM

	F2104098: D8968.id30426.diff
	Tue, Jun 25, 4:21 AM

	F2104030: D8968.id.diff
	Tue, Jun 25, 4:20 AM

	F2103965: D8968.diff
	Tue, Jun 25, 4:12 AM

	Unknown Object (File)
	Mon, Jun 24, 11:13 PM

	Unknown Object (File)
	Thu, Jun 20, 9:57 AM

	Unknown Object (File)
	Sat, Jun 15, 8:12 AM

View All 35 Files

Subscribers

ashoat

tomek

Details

Reviewers

michal
• patryk
varun
• jon

Commits

rCOMMd794c76960af: [reports-service] Add service error types

Summary

Adds service error types to be used by the reports service and conversions to HTTP error responses.

Depends on D8951

Test Plan

Tested together with next diffs

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bartek created this revision.Aug 28 2023, 2:38 AM

bartek held this revision as a draft.

Herald added a reviewer: • jon. · View Herald TranscriptAug 28 2023, 2:38 AM

Herald added subscribers: tomek, ashoat. · View Herald Transcript

bartek added a child revision: D8970: [reports-service] Endpoint for uploading reports.Aug 28 2023, 2:44 AM

Harbormaster completed remote builds in B22160: Diff 30426.Aug 28 2023, 2:54 AM

bartek published this revision for review.Aug 28 2023, 2:57 AM

michal accepted this revision.Aug 28 2023, 4:16 AM

This revision is now accepted and ready to land.Aug 28 2023, 4:16 AM

Closed by commit rCOMMd794c76960af: [reports-service] Add service error types (authored by bartek). · Explain WhyAug 29 2023, 8:04 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMMd794c76960af: [reports-service] Add service error types.

Revision Contents
Changeset List

Path

Size

services/

reports/

src/

http.rs

69 lines

service.rs

30 lines

Diff 30510

View Options

services/reports/src/http.rs