aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorQuentin Dufour <quentin@deuxfleurs.fr>2022-06-13 10:13:31 +0200
committerQuentin Dufour <quentin@deuxfleurs.fr>2022-06-13 10:19:52 +0200
commitd544a0e0e03c9b69b226fb5bba2ce27a7af270ca (patch)
tree287b4e735756f1bca3e4942a50d5bd62f13e159c
parent138e13071be37d873344cd03e316c87ff8057ea0 (diff)
downloadgarage-bug/cors.tar.gz
garage-bug/cors.zip
Send CORS headers for all requestsbug/cors
-rw-r--r--src/api/s3/api_server.rs11
1 files changed, 2 insertions, 9 deletions
diff --git a/src/api/s3/api_server.rs b/src/api/s3/api_server.rs
index ecc417ab..d1d6288c 100644
--- a/src/api/s3/api_server.rs
+++ b/src/api/s3/api_server.rs
@@ -4,7 +4,7 @@ use async_trait::async_trait;
use futures::future::Future;
use hyper::header;
-use hyper::{Body, Method, Request, Response};
+use hyper::{Body, Request, Response};
use opentelemetry::{trace::SpanRef, KeyValue};
@@ -167,14 +167,7 @@ impl ApiHandler for S3ApiServer {
return Err(Error::forbidden("Operation is not allowed for this key."));
}
- // Look up what CORS rule might apply to response.
- // Requests for methods different than GET, HEAD or POST
- // are always preflighted, i.e. the browser should make
- // an OPTIONS call before to check it is allowed
- let matching_cors_rule = match *req.method() {
- Method::GET | Method::HEAD | Method::POST => find_matching_cors_rule(&bucket, &req)?,
- _ => None,
- };
+ let matching_cors_rule = find_matching_cors_rule(&bucket, &req)?;
let resp = match endpoint {
Endpoint::HeadObject {