Skip to content

Commit 0429cd7

Browse files
CLDSRV-636: Migrate SSE on object(Get|Head|Put)
1 parent e4adb8f commit 0429cd7

File tree

3 files changed

+9
-3
lines changed

3 files changed

+9
-3
lines changed

lib/api/objectGet.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ const { metadataValidateBucketAndObj } = require('../metadata/metadataUtils');
1313
const monitoring = require('../utilities/metrics');
1414
const { getPartCountFromMd5 } = require('./apiUtils/object/partInfo');
1515
const { setExpirationHeaders } = require('./apiUtils/object/expirationHeaders');
16+
const { updateEncryption } = require('./apiUtils/bucket/updateEncryption');
1617

1718
const validateHeaders = s3middleware.validateConditionalHeaders;
1819

@@ -51,6 +52,7 @@ function objectGet(authInfo, request, returnTagCount, log, callback) {
5152
};
5253

5354
return metadataValidateBucketAndObj(mdValParams, log,
55+
(err, bucket, objMD) => updateEncryption(err, bucket, objMD, objectKey, log, {},
5456
(err, bucket, objMD) => {
5557
const corsHeaders = collectCorsHeaders(request.headers.origin,
5658
request.method, bucket);
@@ -255,7 +257,7 @@ function objectGet(authInfo, request, returnTagCount, log, callback) {
255257
return callback(null, dataLocator, responseMetaHeaders,
256258
byteRange);
257259
});
258-
});
260+
}));
259261
}
260262

261263
module.exports = objectGet;

lib/api/objectHead.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ const { getPartNumber, getPartSize, getPartCountFromMd5 } =
1414
const { metadataValidateBucketAndObj } = require('../metadata/metadataUtils');
1515
const { maximumAllowedPartCount } = require('../../constants');
1616
const { setExpirationHeaders } = require('./apiUtils/object/expirationHeaders');
17+
const { updateEncryption } = require('./apiUtils/bucket/updateEncryption');
1718

1819
/**
1920
* HEAD Object - Same as Get Object but only respond with headers
@@ -51,6 +52,7 @@ function objectHead(authInfo, request, log, callback) {
5152
};
5253

5354
return metadataValidateBucketAndObj(mdValParams, log,
55+
(err, bucket, objMD) => updateEncryption(err, bucket, objMD, objectKey, log, {},
5456
(err, bucket, objMD) => {
5557
const corsHeaders = collectCorsHeaders(request.headers.origin,
5658
request.method, bucket);
@@ -160,7 +162,7 @@ function objectHead(authInfo, request, log, callback) {
160162
});
161163
monitoring.promMetrics('HEAD', bucketName, '200', 'headObject');
162164
return callback(null, responseHeaders);
163-
});
165+
}));
164166
}
165167

166168
module.exports = objectHead;

lib/api/objectPut.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ const validateChecksumHeaders = require('./apiUtils/object/validateChecksumHeade
1919

2020
const writeContinue = require('../utilities/writeContinue');
2121
const versionIdUtils = versioning.VersionID;
22+
const { updateEncryption } = require('./apiUtils/bucket/updateEncryption');
2223

2324
/**
2425
* PUT Object in the requested bucket. Steps include:
@@ -79,6 +80,7 @@ function objectPut(authInfo, request, streamingV4Params, log, callback) {
7980
log.trace('owner canonicalID to send to data', { canonicalID });
8081

8182
return metadataValidateBucketAndObj(valParams, log,
83+
(err, bucket, objMD) => updateEncryption(err, bucket, objMD, objectKey, log, { skipObject: true },
8284
(err, bucket, objMD) => {
8385
const responseHeaders = collectCorsHeaders(headers.origin,
8486
method, bucket);
@@ -202,7 +204,7 @@ function objectPut(authInfo, request, streamingV4Params, log, callback) {
202204
null, ingestSize);
203205
return callback(null, responseHeaders);
204206
});
205-
});
207+
}));
206208
}
207209

208210
module.exports = objectPut;

0 commit comments

Comments
 (0)