Paths

Table of Contentst

Differential D8881

[services] Extract cors config to services-lib
ClosedPublic
Actions

Authored by michal on Aug 21 2023, 9:07 AM.

Tags

None

Referenced Files

	F33274528: D8881.1768658944.diff
	Fri, Jan 16, 2:09 PM

	F33269065: D8881.1768634087.diff
	Fri, Jan 16, 7:14 AM

Subscribers

Details

Reviewers

bartek
kamil
• patryk
• jon

Commits

rCOMM6fcda8c904f5: [services] Extract cors config to services-lib

Summary

ENG-4501
Backup service will be now a http servce so it will need a cors config. This diff extracts it from blob to services lib.

Depends on D8869

Test Plan

cargo build in blob

Diff Detail

Repository

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

michal created this revision.Aug 21 2023, 9:07 AM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptAug 21 2023, 9:07 AM

michal added inline comments.Aug 21 2023, 9:10 AM

services/blob/Cargo.toml
26–32 ↗	(On Diff #30127)	VSCode automatically formatted these lines. They are a bit more consistent now so I included it in this diff
services/comm-services-lib/src/lib.rs
4–5 ↗	(On Diff #30127)	Added a new module/ feature for http related things

michal added a child revision: D8882: [backup] Basic http server.Aug 21 2023, 9:13 AM

Harbormaster completed remote builds in B21935: Diff 30127.Aug 21 2023, 9:25 AM

michal requested review of this revision.Aug 21 2023, 9:25 AM

Rebase

Harbormaster completed remote builds in B21956: Diff 30153.Aug 21 2023, 4:59 PM

• jon accepted this revision.Aug 21 2023, 7:26 PM

• jon added inline comments.

services/comm-services-lib/src/lib.rs
3–4	Nice, like the feature-opt-in aspect of this.

This revision is now accepted and ready to land.Aug 21 2023, 7:26 PM

bartek accepted this revision.Aug 22 2023, 7:14 AM

Closed by commit rCOMM6fcda8c904f5: [services] Extract cors config to services-lib. · Explain WhyAug 22 2023, 10:21 AM

This revision was automatically updated to reflect the committed changes.

Restricted Repository Identity added a commit: rCOMM6fcda8c904f5: [services] Extract cors config to services-lib.

Revision Contents
Changeset List

Path

Size

services/

backup/

blob/

9 lines

src/

http/

22 lines

comm-services-lib/

3 lines

src/

18 lines

2 lines

Diff 30153

services/backup/Cargo.lock

Loading...

services/blob/Cargo.lock

Loading...

services/blob/Cargo.toml

Loading...

services/blob/src/http/mod.rs

Loading...

services/comm-services-lib/Cargo.lock

Loading...

services/comm-services-lib/Cargo.toml

Loading...

services/comm-services-lib/src/http.rs

Loading...

services/comm-services-lib/src/lib.rs

Loading...