Fix the semver version check for adminPassword props

Signed-off-by: Sayali Gaikawad <gaiksaya@amazon.com>

lib/infra/infra-stack.ts

@@ -38,6 +38,7 @@ import { LogGroup, RetentionDays } from 'aws-cdk-lib/aws-logs';
 import { readFileSync } from 'fs';
 import { dump, load } from 'js-yaml';
 import { join } from 'path';
+import { satisfies } from 'semver';
 import { CloudwatchAgent } from '../cloudwatch/cloudwatch-agent';
 import { ProcstatMetricDefinition } from '../cloudwatch/metrics-section';
 import { InfraStackMonitoring } from '../monitoring/alarms';
@@ -206,7 +207,7 @@ export class InfraStack extends Stack {
     this.securityDisabled = securityDisabled === 'true';
 
     this.adminPassword = this.securityDisabled ? '' : `${props?.adminPassword ?? scope.node.tryGetContext('adminPassword')}`;
-    if (!this.securityDisabled && Number.parseFloat(this.distVersion) >= 2.12 && this.adminPassword === 'undefined') {
+    if (!this.securityDisabled && satisfies(this.distVersion, '>=2.12.0') && this.adminPassword === 'undefined') {
       throw new Error('adminPassword parameter is required to be set when security is enabled');
     }
 

package.json

@@ -13,9 +13,10 @@
   },
   "devDependencies": {
     "@types/jest": "^27.5.2",
+    "@types/js-yaml": "^4.0.5",
     "@types/node": "10.17.27",
     "@types/prettier": "2.6.0",
-    "@types/js-yaml": "^4.0.5",
+    "@types/semver": "^7.5.6",
     "aws-cdk": "2.45.0",
     "jest": "^27.5.1",
     "ts-jest": "^27.1.4",
@@ -27,10 +28,11 @@
     "@typescript-eslint/parser": "^4.31.1",
     "aws-cdk-lib": "2.45.0",
     "constructs": "^10.0.0",
-    "js-yaml": "^4.1.0",
-    "source-map-support": "^0.5.21",
     "eslint": "^7.32.0",
     "eslint-config-airbnb-base": "^14.2.1",
-    "eslint-plugin-import": "^2.24.2"
+    "eslint-plugin-import": "^2.24.2",
+    "js-yaml": "^4.1.0",
+    "semver": "^7.5.6",
+    "source-map-support": "^0.5.21"
   }
 }