Skip to content

Commit

Permalink
logs
Browse files Browse the repository at this point in the history
  • Loading branch information
benzekrimaha committed Jul 11, 2024
1 parent 75b8b14 commit 75da000
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion lib/api/multiObjectDelete.js
Original file line number Diff line number Diff line change
Expand Up @@ -571,8 +571,9 @@ function multiObjectDelete(authInfo, request, log, callback) {
};
return vault.checkPolicies(requestContextParams, authInfo.getArn(),
log, (err, authorizationResults) => {
log.info('received authorization results', authorizationResults);
log.info('received authorization results', { authorizationResults });
// there were no policies so received a blanket AccessDenied
log.info("checking error", { err });

Check warning on line 576 in lib/api/multiObjectDelete.js

View workflow job for this annotation

GitHub Actions / linting-coverage

Strings must use singlequote
if (err?.is.AccessDenied) {
objects.forEach(entry => {
errorResults.push({
Expand All @@ -589,6 +590,8 @@ function multiObjectDelete(authInfo, request, log, callback) {
});
return next(err);
}
log.info('authorization results', { authorizationResults });
log.info("checking condition", { condition: objects.length !== authorizationResults.length })

Check warning on line 594 in lib/api/multiObjectDelete.js

View workflow job for this annotation

GitHub Actions / linting-coverage

Strings must use singlequote

Check failure on line 594 in lib/api/multiObjectDelete.js

View workflow job for this annotation

GitHub Actions / linting-coverage

Missing semicolon
if (objects.length !== authorizationResults.length) {
log.error('vault did not return correct number of ' +
'authorization results', {
Expand All @@ -605,6 +608,7 @@ function multiObjectDelete(authInfo, request, log, callback) {
acc[apiMethod] = curr.isImplicit;
return acc;
}, {});
log.info("actionImplicitDenies", { actionImplicitDenies });

Check warning on line 611 in lib/api/multiObjectDelete.js

View workflow job for this annotation

GitHub Actions / linting-coverage

Strings must use singlequote
for (let i = 0; i < authorizationResults.length; i++) {
const result = authorizationResults[i];
log.info('checking authorization result', result);
Expand Down

0 comments on commit 75da000

Please sign in to comment.