Skip to content

*: Allow to disable automatically setting CORS headers#3919

Merged
onprem merged 1 commit intothanos-io:mainfrom
brancz:disable-cors
Mar 11, 2021
Merged

*: Allow to disable automatically setting CORS headers#3919
onprem merged 1 commit intothanos-io:mainfrom
brancz:disable-cors

Conversation

@brancz
Copy link
Copy Markdown
Member

@brancz brancz commented Mar 11, 2021

  • I added CHANGELOG entry for this change.
  • Change is not relevant to the end user.

Changes

Add --web.disable-cors flag to every component that exposes an API, allowing to disable automatically setting CORS headers.

Verification

Ran components with and without the flag using the scripts/quickstart.sh script and verified the headers are gone.

Closes #3918

@thanos-io/thanos-maintainers

Signed-off-by: Frederic Branczyk <fbranczyk@gmail.com>
Copy link
Copy Markdown
Member

@onprem onprem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@onprem onprem merged commit b2474aa into thanos-io:main Mar 11, 2021
@brancz brancz deleted the disable-cors branch March 12, 2021 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow disabling setting CORS headers

2 participants