Skip to content

Commit f485429

Browse files
iNecasdekobon
authored andcommitted
Re-apply fix auth v4 for directory listing
I was getting `SignatureDoesNotMatch` errors when using directory listing with v4 authentication. Seems like the problem was the requests were sent with trailing `/` while the signature was calculated without it. It doesn't seem we need the trailing space there (I have checked only with Ceph S3 API, so more testing with AWS welcome), but seems like the code should be broken for every back-end, unless I'm missing something.
1 parent a6e0aa4 commit f485429

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

common/etc/nginx/include/s3gateway.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -352,7 +352,7 @@ function s3uri(r) {
352352
if (allow_listing) {
353353
var queryParams = _s3DirQueryParams(uriPath, r.method);
354354
if (queryParams.length > 0) {
355-
path = basePath + '/?' + queryParams;
355+
path = basePath + '?' + queryParams;
356356
} else {
357357
path = basePath + uriPath;
358358
}

0 commit comments

Comments
 (0)