search

drogue-iot / drogue-cloud

Cloud Native IoT
https://drogue.io
Apache License 2.0
113 stars 30 forks source link

add CORS to http endpoint #341

Closed jbtrystram closed 1 year ago

jbtrystram commented 1 year ago

I am not sure what origin (if any) we should allow by default here ?

jbtrystram commented 1 year ago

This depends on https://github.com/drogue-iot/drogue-bazaar/pull/3

jbtrystram commented 1 year ago

/test

drogue-bot commented 1 year ago

Aye, aye, captain! … Triggered System Test worklow!

drogue-bot commented 1 year ago
Total Passed Failed Ignored Filtered Duration
❌ 148 147 1 0 0 31m 13s

Git: https://github.com/drogue-iot/drogue-cloud @ refs/heads/cors

Commit: 3b8ca60e9842bbfd4b72096956895b3788446999
Author: jbtrystram <jbtrystram@redhat.com>
Date: Mon, 07 Nov 2022 14:20:44 -0100

    readd a removed dependency. oops

Job: https://github.com/drogue-iot/drogue-cloud-testing/actions/runs/3420569921

Report: https://drogue-iot.github.io/drogue-cloud-testing/test-report/2022/11/08/test-run-3420569921.html

ctron commented 1 year ago

I think you need to rebase the PR