Skip to content

Commit c8e4647

Browse files
author
Will Toozs
committed
update response headers
1 parent 95140e3 commit c8e4647

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

lib/api/objectPost.js

+1-2
Original file line numberDiff line numberDiff line change
@@ -95,9 +95,8 @@ function objectPost(authInfo, request, streamingV4Params, log, callback) {
9595
});
9696
if (storingResult) {
9797
// ETag's hex should always be enclosed in quotes
98-
responseHeaders.location = `/${bucketName}/${encodeURIComponent(request.formData.key)}`;
99-
responseHeaders.Bucket = bucketName;
10098
responseHeaders.ETag = `"${storingResult.contentMD5}"`;
99+
responseHeaders.location = `${request.headers.host}/${bucketName}/${encodeURIComponent(request.formData.key)}`;
101100
}
102101
const vcfg = bucket.getVersioningConfiguration();
103102
const isVersionedObj = vcfg && vcfg.Status === 'Enabled';

tests/functional/aws-node-sdk/test/object/post.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -185,7 +185,7 @@ describe('POST object', () => {
185185
req.on('response', res => {
186186
try {
187187
assert.equal(res.statusCode, 204);
188-
assert.equal(res.headers.location, `/${bucketName}/${filename}`);
188+
assert.equal(res.headers.location, `${url.hostname}:${url.port}/${bucketName}/${filename}`);
189189
done();
190190
} catch (err) {
191191
done(err);
@@ -234,7 +234,7 @@ describe('POST object', () => {
234234
req.on('response', res => {
235235
try {
236236
assert.equal(res.statusCode, 204);
237-
assert.equal(res.headers.location, `/${bucketName}/${encodedKey}`);
237+
assert.equal(res.headers.location, `${url.hostname}:${url.port}/${bucketName}/${encodedKey}`);
238238
done();
239239
} catch (err) {
240240
done(err);

0 commit comments

Comments
 (0)