csv: fix values for policy and acl fields being the opposite of what …

…the heading suggested
scalefactory · Oct 4, 2019 · e3bff3f · e3bff3f
1 parent 3dff1d3
commit e3bff3f
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/formatters/csv.ts b/src/formatters/csv.ts
@@ -128,7 +128,7 @@ export default class Csv implements S3Audit.Types.Formatter {
     return new Promise(resolve => {
       bucket.getPolicyWildcardEntities()
         .then((entities: Array<string>) => {
-          bucketDetails.bucket_policy = entities.length > 0
+          bucketDetails.bucket_policy = entities.length === 0
         })
         .catch((error: AWSError) => {
           bucketDetails.bucket_policy = error.message
@@ -140,8 +140,8 @@ export default class Csv implements S3Audit.Types.Formatter {
   private populateACLField(bucket: Bucket, bucketDetails: S3Audit.Types.CSVFields) {
     return new Promise(resolve => {
       bucket.allowsPublicAccessViaACL()
-        .then((isAllowed: boolean) => {
-          bucketDetails.bucket_acl = isAllowed
+        .then((isPublic: boolean) => {
+          bucketDetails.bucket_acl = isPublic === false
         })
         .catch((error: AWSError) => {
           bucketDetails.bucket_acl = error.message