diff --git a/src/leo_s3_auth.erl b/src/leo_s3_auth.erl index 170928d..c014a03 100644 --- a/src/leo_s3_auth.erl +++ b/src/leo_s3_auth.erl @@ -860,7 +860,8 @@ auth_sub_resources(QueryStr) -> -ifdef(TEST). auth_uri_test() -> Bucket = <<"photo">>, - <<"">> = auth_uri(Bucket, <<"/photo">>, <<"/photo">>), + %% For bucket operations, AWS S3 canonical resource ends with "/" (e.g., /bucket/) + <<"/">> = auth_uri(Bucket, <<"/photo">>, <<"/photo">>), <<"/photo">> = auth_uri(Bucket, <<"/photo">>, <<"/photo/photo">>), <<"/">> = auth_uri(Bucket, <<"/photo/">>, <<"/photo/">>), diff --git a/src/leo_s3_bucket.erl b/src/leo_s3_bucket.erl index 6e8f7c9..fba2e21 100644 --- a/src/leo_s3_bucket.erl +++ b/src/leo_s3_bucket.erl @@ -65,13 +65,7 @@ start(slave = Type, Provider, SyncInterval) -> catch ets:new(?BUCKET_TABLE, [named_table, ordered_set, public, {read_concurrency, true}]), catch ets:new(?BUCKET_INFO, [named_table, set, public, {read_concurrency, true}]), - - case Provider of - [] -> - void; - _ -> - ok = setup(Type, ets, Provider, SyncInterval) - end, + ok = setup(Type, ets, Provider, SyncInterval), ok; start(master = Type, _Provider, SyncInterval) ->