Merge branches 'w/8.1/feature/S3C-4614/assumerole' and 'q/1495/7.10/f…

…eature/S3C-4614/assumerole' into tmp/octopus/q/8.1
scality · Jun 29, 2021 · 16a5e6a · 16a5e6a
3 parents 15703aa + 864d2e8 + 5540afa
commit 16a5e6a
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/lib/policyEvaluator/principal.js b/lib/policyEvaluator/principal.js
@@ -143,6 +143,7 @@ class Principal {
                 AWS: [
                     account,
                     accountArn,
+                    requesterArn,
                 ],
             };
             checkAction = true;

diff --git a/package.json b/package.json
@@ -3,7 +3,7 @@
   "engines": {
     "node": ">=8"
   },
-  "version": "8.2.1",
+  "version": "8.2.2",
   "description": "Common utilities for the S3 project components",
   "main": "index.js",
   "repository": {

diff --git a/tests/unit/policy/test_principalEvaluator.js b/tests/unit/policy/test_principalEvaluator.js
@@ -472,7 +472,7 @@ describe('Principal evaluator', () => {
             },
         },
         {
-            name: 'should deny user as principal if account is different',
+            name: 'should allow user as principal if account is different',
             statement: [
                 {
                     Principal: {
@@ -491,7 +491,7 @@ describe('Principal evaluator', () => {
                 accountId: defaultAccountId,
             },
             result: {
-                result: 'Deny',
+                result: 'Allow',
                 checkAction: true,
             },
         },