Skip to content

Commit dfa80a4

Browse files
author
Will Toozs
committed
update variable name
1 parent 9f753af commit dfa80a4

33 files changed

+120
-96
lines changed

lib/api/bucketPutACL.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ function bucketPutACL(authInfo, request, log, callback) {
8686

8787
return async.waterfall([
8888
function waterfall1(next) {
89-
metadataValidateBucket(metadataValParams, request.iamAuthzResults, log,
89+
metadataValidateBucket(metadataValParams, request.actionImplicitDenies, log,
9090
(err, bucket) => {
9191
if (err) {
9292
log.trace('request authorization failed', {

lib/api/bucketPutCors.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ function bucketPutCors(authInfo, request, log, callback) {
6666
},
6767
function validateBucketAuthorization(bucket, rules, corsHeaders, next) {
6868
if (!isBucketAuthorized(bucket, request.apiMethods || requestType, canonicalID, authInfo,
69-
request.iamAuthzResults, log, request)) {
69+
request.actionImplicitDenies, log, request)) {
7070
log.debug('access denied for account on bucket', {
7171
requestType,
7272
});

lib/api/bucketPutEncryption.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ function bucketPutEncryption(authInfo, request, log, callback) {
2828
};
2929

3030
return async.waterfall([
31-
next => metadataValidateBucket(metadataValParams, request.iamAuthzResults, log, next),
31+
next => metadataValidateBucket(metadataValParams, request.actionImplicitDenies, log, next),
3232
(bucket, next) => checkExpectedBucketOwner(request.headers, bucket, log, err => next(err, bucket)),
3333
(bucket, next) => {
3434
log.trace('parsing encryption config', { method: 'bucketPutEncryption' });

lib/api/bucketPutLifecycle.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ function bucketPutLifecycle(authInfo, request, log, callback) {
4141
return next(null, configObj);
4242
});
4343
},
44-
(lcConfig, next) => metadataValidateBucket(metadataValParams, request.iamAuthzResults, log,
44+
(lcConfig, next) => metadataValidateBucket(metadataValParams, request.actionImplicitDenies, log,
4545
(err, bucket) => {
4646
if (err) {
4747
return next(err, bucket);

lib/api/bucketPutNotification.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ function bucketPutNotification(authInfo, request, log, callback) {
3434
const notifConfig = notificationConfig.error ? undefined : notificationConfig;
3535
process.nextTick(() => next(notificationConfig.error, notifConfig));
3636
},
37-
(notifConfig, next) => metadataValidateBucket(metadataValParams, request.iamAuthzResults, log,
37+
(notifConfig, next) => metadataValidateBucket(metadataValParams, request.actionImplicitDenies, log,
3838
(err, bucket) => next(err, bucket, notifConfig)),
3939
(bucket, notifConfig, next) => {
4040
bucket.setNotificationConfiguration(notifConfig);

lib/api/bucketPutObjectLock.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ function bucketPutObjectLock(authInfo, request, log, callback) {
4141
return next(configObj.error || null, configObj);
4242
});
4343
},
44-
(objectLockConfig, next) => metadataValidateBucket(metadataValParams, request.iamAuthzResults,
44+
(objectLockConfig, next) => metadataValidateBucket(metadataValParams, request.actionImplicitDenies,
4545
log, (err, bucket) => {
4646
if (err) {
4747
return next(err, bucket);

lib/api/bucketPutPolicy.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ function bucketPutPolicy(authInfo, request, log, callback) {
6969
return next(null, bucketPolicy);
7070
});
7171
},
72-
(bucketPolicy, next) => metadataValidateBucket(metadataValParams, request.iamAuthzResults, log,
72+
(bucketPolicy, next) => metadataValidateBucket(metadataValParams, request.actionImplicitDenies, log,
7373
(err, bucket) => {
7474
if (err) {
7575
return next(err, bucket);

lib/api/bucketPutReplication.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ function bucketPutReplication(authInfo, request, log, callback) {
3636
// Check bucket user privileges and ensure versioning is 'Enabled'.
3737
(config, next) =>
3838
// TODO: Validate that destination bucket exists and has versioning.
39-
metadataValidateBucket(metadataValParams, request.iamAuthzResults, log, (err, bucket) => {
39+
metadataValidateBucket(metadataValParams, request.actionImplicitDenies, log, (err, bucket) => {
4040
if (err) {
4141
return next(err);
4242
}

lib/api/bucketPutVersioning.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ function bucketPutVersioning(authInfo, request, log, callback) {
9393

9494
return waterfall([
9595
next => _parseXML(request, log, next),
96-
next => metadataValidateBucket(metadataValParams, request.iamAuthzResults, log,
96+
next => metadataValidateBucket(metadataValParams, request.actionImplicitDenies, log,
9797
(err, bucket) => next(err, bucket)), // ignore extra null object,
9898
(bucket, next) => parseString(request.post, (err, result) => {
9999
// just for linting; there should not be any parsing error here

lib/api/bucketPutWebsite.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ function bucketPutWebsite(authInfo, request, log, callback) {
4747
},
4848
function validateBucketAuthorization(bucket, config, next) {
4949
if (!isBucketAuthorized(bucket, request.apiMethods || requestType, canonicalID, authInfo,
50-
request.iamAuthzResults, log, request)) {
50+
request.actionImplicitDenies, log, request)) {
5151
log.debug('access denied for user on bucket', {
5252
requestType,
5353
method: 'bucketPutWebsite',

0 commit comments

Comments
 (0)