snarkOS icon indicating copy to clipboard operation
snarkOS copied to clipboard

[TOB] allow the restriction of CORS origins

Open ljedrz opened this issue 1 year ago • 2 comments

Cc @zosorock, as this is probably most applicable to services run atop the REST server.

Finding: TOB-ALEO-23

ljedrz avatar Nov 07 '23 11:11 ljedrz

Alternatively, instead of this PR we could remove the direct CORS support from REST API and let node operators run a reverse proxy for snarkOS in which the CORS is configured (e.g. nginx), if they need it.

This option would simplify our codebase and allow more flexible CORS configurations, but it would also add that extra step and operational overhead when integrating snarkOS into websites via REST (unsure how common use case that'll be).

vvp avatar Nov 08 '23 14:11 vvp

This is already guarded using firewalls today.

howardwu avatar Nov 10 '23 00:11 howardwu