The context is here: https://linear.app/comm/issue/ENG-8806/unable-to-call-identity-http-endpoint-from-service-workerjs-due-to. There were two CORS issues with identity HTTP. This differential applies two fixes from Varun and Will combined
Details
Details
- Reviewers
varun will bartek - Commits
- rCOMM52aa37541e68: Fix CORS issues with identity HTTP
Execute test plan for https://phab.comm.dev/D12703 (D12703). Without changes in this diff it is not possible to succesfully fetch inbound keys from service worker.
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Branch
- marcin/eng-8806
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
| services/identity/Cargo.toml | ||
|---|---|---|
| 45 | @varun Should we normalize adding cargo crates to the cargo workspace Cargo.toml? | |
| services/identity/src/constants.rs | ||
| 313 | So besides for authorization, accept is also required? | |
| services/identity/src/websockets/mod.rs | ||
| 116 | We can probably remove this comment as the code is self-explanatory | |
| services/identity/Cargo.toml | ||
|---|---|---|
| 45 | yeah good call-out. we should add this in the root Cargo.toml instead | |
| services/identity/src/constants.rs | ||
|---|---|---|
| 313 | It is possible that it is not necessary but I will need a test to confirm. | |
| services/identity/src/constants.rs | ||
|---|---|---|
| 313 | Confirming that accept header is not necessary | |