fixed

aquasecurity · Dec 1, 2023 · e39df97 · e39df97
1 parent 6e16236
commit e39df97
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 8 deletions.
diff --git a/exports.js b/exports.js
@@ -806,7 +806,7 @@ module.exports = {
         'logRetentionDays'              : require(__dirname + '/plugins/azure/postgresqlserver/logRetentionDays.js'),
         'connectionThrottlingEnabled'   : require(__dirname + '/plugins/azure/postgresqlserver/connectionThrottlingEnabled.js'),
         'logDurationEnabled'            : require(__dirname + '/plugins/azure/postgresqlserver/logDurationEnabled.js'),
-        'postgresqlCMKEncrypted'        : require(__dirname + '/plugins/azure/postgresqlserver/postgresqlEncryptionAtRestWithCMK.js'),
+        'postgresqlCMKEncrypted'        : require(__dirname + '/plugins/azure/postgresqlserver/postgresqlCMKEncrypted.js'),
         'logDisconnectionsEnabled'      : require(__dirname + '/plugins/azure/postgresqlserver/logDisconnectionsEnabled.js'),
         'logConnectionsEnabled'         : require(__dirname + '/plugins/azure/postgresqlserver/logConnectionsEnabled.js'),
         'logCheckpointsEnabled'         : require(__dirname + '/plugins/azure/postgresqlserver/logCheckpointsEnabled.js'),

diff --git a/...rver/postgresqlEncryptionAtRestWithCMK.js → ...ostgresqlserver/postgresqlCMKEncrypted.js b/...rver/postgresqlEncryptionAtRestWithCMK.js → ...ostgresqlserver/postgresqlCMKEncrypted.js
@@ -35,7 +35,7 @@ module.exports = {
 
             for (let server of servers.data) {
                 if (!server.id) continue;
-                if (server.byokEnforcement && server.byokEnforcement == 'Enabled') {
+                if (server.byokEnforcement && server.byokEnforcement.toLowerCase() === 'enabled') {
                     helpers.addResult(results, 0, 'PostgreSQL server is encrypted using CMK', location, server.id);
                 } else {
                     helpers.addResult(results, 2, 'PostgreSQL server is not encrypted using CMK', location, server.id);

diff --git a/...postgresqlEncryptionAtRestWithCMK.spec.js → ...esqlserver/postgresqlCMKEncrypted.spec.js b/...postgresqlEncryptionAtRestWithCMK.spec.js → ...esqlserver/postgresqlCMKEncrypted.spec.js
@@ -1,5 +1,5 @@
 var expect = require('chai').expect;
-var postgresqlEncryptionAtRestWithCMK = require('./postgresqlEncryptionAtRestWithCMK');
+var postgresqlCMKEncrypted = require('./postgresqlCMKEncrypted');
 
 const listPostgres = [
     {
@@ -80,11 +80,11 @@ const createCache = (listPostgres) => {
     };
 };
 
-describe('postgresqlEncryptionAtRestWithCMK', function() {
+describe('postgresqlCMKEncrypted', function() {
     describe('run', function() {
         it('should give passing result if no servers', function(done) {
             const cache = createCache({});
-            postgresqlEncryptionAtRestWithCMK.run(cache, {}, (err, results) => {
+            postgresqlCMKEncrypted.run(cache, {}, (err, results) => {
                 expect(results.length).to.equal(1);
                 expect(results[0].status).to.equal(0);
                 expect(results[0].message).to.include('No existing PostgreSQL Servers found');
@@ -95,7 +95,7 @@ describe('postgresqlEncryptionAtRestWithCMK', function() {
 
         it('should give failing result if PostgreSQL Server is not encrypted using CMK', function(done) {
             const cache = createCache([listPostgres[0]]);
-            postgresqlEncryptionAtRestWithCMK.run(cache, {}, (err, results) => {
+            postgresqlCMKEncrypted.run(cache, {}, (err, results) => {
                 expect(results.length).to.equal(1);
                 expect(results[0].status).to.equal(2);
                 expect(results[0].message).to.include('PostgreSQL server is not encrypted using CMK');
@@ -106,7 +106,7 @@ describe('postgresqlEncryptionAtRestWithCMK', function() {
 
         it('should give passing result if PostgreSQL Server is encrypted using CMK', function(done) {
             const cache = createCache([listPostgres[1]]);
-            postgresqlEncryptionAtRestWithCMK.run(cache, {}, (err, results) => {
+            postgresqlCMKEncrypted.run(cache, {}, (err, results) => {
                 expect(results.length).to.equal(1);
                 expect(results[0].status).to.equal(0);
                 expect(results[0].message).to.include('PostgreSQL server is encrypted using CMK');
@@ -116,7 +116,7 @@ describe('postgresqlEncryptionAtRestWithCMK', function() {
         });
         it('should give UnKnown result if unable to query postgreSQL Server', function(done) {
             const cache = createCache(null);
-            postgresqlEncryptionAtRestWithCMK.run(cache, {}, (err, results) => {
+            postgresqlCMKEncrypted.run(cache, {}, (err, results) => {
                 expect(results.length).to.equal(1);
                 expect(results[0].status).to.equal(3);
                 expect(results[0].message).to.include('Unable to query for PostgreSQL Servers:');