@ndlib/ndlib-cdk2 1.0.36 → 1.0.38

package/README.md

@@ -113,7 +113,7 @@ if(props.slackChannelId){
     // messageText: 'example of additional message text', // optional
     slackChannelId: props.slackChannelId,
     slackChannelName: props.slackChannelName,
-    // slackNotifyTopicArn: 'some valid topic arn', // optional, defaults to: Fn.importValue('slack-pipeline-status-notify:TopicArn')
+    slackNotifyTopicArn: 'some valid topic arn', 
   })
 }
 ```
@@ -424,8 +424,8 @@ import { Topic } from 'aws-cdk-lib/aws-sns'
 
 ...
 // Create a Topic
-    const importedSlackNotifyTopicArn = Fn.importValue(props.slackNotifyTopicOutput)
-    const approvalTopic = Topic.fromTopicArn(this, 'SlackTopicFromArn', importedSlackNotifyTopicArn)
+    const slackNotifyTopicArn = StringParameter.valueForStringParameter(this, props.slackNotifyTopicArnParameterPath)
+    const approvalTopic = Topic.fromTopicArn(this, 'SlackTopicFromArn', slackNotifyTopicArn)
 
 // Call the SlackIntegratedManualApproval
    const approveTestStackAction = new SlackIntegratedManualApproval({

package/lib/https-alb.js

@@ -23,7 +23,7 @@ class HttpsAlb extends aws_cdk_lib_1.aws_elasticloadbalancingv2.ApplicationLoadB
             open: true,
             port: 443,
             protocol: aws_cdk_lib_1.aws_elasticloadbalancingv2.ApplicationProtocol.HTTPS,
-            sslPolicy: aws_cdk_lib_1.aws_elasticloadbalancingv2.SslPolicy.TLS13_EXT2,
+            sslPolicy: aws_cdk_lib_1.aws_elasticloadbalancingv2.SslPolicy.TLS13_RES,
         });
         this.defaultListener.addAction('Default404', { action: aws_cdk_lib_1.aws_elasticloadbalancingv2.ListenerAction.fixedResponse(404) });
         // Adding an output of the dns name for convenience when looking at this

package/lib/internal-lambdas/sourceWatcherLambda/src/package.json

@@ -20,12 +20,12 @@
     "node": ">=22.0.0"
   },
   "devDependencies": {
-    "@aws-sdk/client-codepipeline": "^3.835.0",
-    "@aws-sdk/client-ssm": "^3.835.0"
+    "@aws-sdk/client-codepipeline": "^3.840.0",
+    "@aws-sdk/client-ssm": "^3.840.0"
   },
   "dependencies": {
-    "@aws-sdk/client-codepipeline": "^3.835.0",
-    "@aws-sdk/client-ssm": "^3.835.0",
+    "@aws-sdk/client-codepipeline": "^3.840.0",
+    "@aws-sdk/client-ssm": "^3.840.0",
     "glob": "^11.0.3",
     "multimatch": "5.0.0",
     "cross-spawn": "^7.0.6"

package/lib/slack-pipeline-status-notifications.d.ts

@@ -5,7 +5,7 @@ export interface ISlackPipelineStatusNotificationsProps extends StackProps {
     readonly messageText?: string;
     readonly slackChannelId: string;
     readonly slackChannelName?: string;
-    readonly slackNotifyTopicArn?: string;
+    readonly slackNotifyTopicArn: string;
 }
 export interface ISlackPipelineStatusNotificationsData {
     eventInfo: string;

package/lib/slack-pipeline-status-notifications.js

@@ -11,8 +11,7 @@ class SlackPipelineStatusNotifications extends constructs_1.Construct {
         const pipelineName = aws_cdk_lib_1.aws_events.EventField.fromPath('$.detail.pipeline');
         const pipelineState = aws_cdk_lib_1.aws_events.EventField.fromPath('$.detail.state');
         const pipelineBaseUrl = aws_cdk_lib_1.Fn.sub('https://${AWS::Region}.console.aws.amazon.com/codepipeline/home?region=${AWS::Region}#/view/');
-        // eslint-disable-next-line max-len
-        const testStackApprovalTopic = aws_sns_1.Topic.fromTopicArn(this, 'SlackTopicFromArn', props.slackNotifyTopicArn || aws_cdk_lib_1.Fn.importValue('slack-pipeline-status-notify:TopicArn'));
+        const testStackApprovalTopic = aws_sns_1.Topic.fromTopicArn(this, 'SlackTopicFromArn', props.slackNotifyTopicArn);
         const SlackPipelineStatusNotificationsData = {
             eventInfo: eventInfo,
             messageText: props.messageText,

package/lib/source-watcher.d.ts

@@ -30,7 +30,8 @@ export interface ISourceWatcherProps {
     /**
      * Stack name for aws-github-webhook. Needed in order to create a webhook on targetRepo.
      */
-    readonly webhookResourceStackName: string;
+    readonly webhookResourceStackName?: string;
+    readonly webhookResourceArnParameterPath?: string;
 }
 export declare class SourceWatcher extends Construct {
     /**

package/lib/source-watcher.js

@@ -3,6 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.SourceWatcher = void 0;
 const aws_cdk_lib_1 = require("aws-cdk-lib");
 const constructs_1 = require("constructs");
+const aws_ssm_1 = require("aws-cdk-lib/aws-ssm");
 const path = require("path");
 class SourceWatcher extends constructs_1.Construct {
     constructor(scope, id, props) {
@@ -55,8 +56,18 @@ class SourceWatcher extends constructs_1.Construct {
             handler: this.lambdaFunction,
             proxy: true,
         });
+        // Added the following code to get the WebhookLambdaArn from either a parameter store path or a stack name.  (Used to be only from stack name)
+        let githubWebhookLambdaArn = '';
+        if (props.webhookResourceArnParameterPath) {
+            githubWebhookLambdaArn = aws_ssm_1.StringParameter.valueForStringParameter(this, props.webhookResourceArnParameterPath);
+        }
+        if (!githubWebhookLambdaArn && props.webhookResourceStackName) {
+            githubWebhookLambdaArn = aws_cdk_lib_1.Fn.importValue(`${props.webhookResourceStackName}:LambdaArn`);
+        }
+        if (!githubWebhookLambdaArn) {
+            throw new Error('You must provide a webhookResourceArnParameterPath or webhookResourceStackName to create a GitHub webhook.');
+        }
         // Finally, create the Webhook on GitHub to route push events to the API!
-        const githubWebhookLambdaArn = aws_cdk_lib_1.Fn.importValue(`${props.webhookResourceStackName}:LambdaArn`);
         const webhookLambda = aws_cdk_lib_1.aws_lambda.Function.fromFunctionAttributes(this, 'GithubWebhookLambda', {
             functionArn: githubWebhookLambdaArn,
             skipPermissions: true,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ndlib/ndlib-cdk2",
-  "version": "1.0.36",
+  "version": "1.0.38",
   "description": "Reusable CDK2 modules used within Hesburgh Libraries of Notre Dame",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",
@@ -37,24 +37,24 @@
   },
   "homepage": "https://github.com/ndlibrary/ndlib-cdk2#readme",
   "peerDependencies": {
-    "aws-cdk-lib": "^2.202.0"
+    "aws-cdk-lib": "^2.206.0"
   },
   "devDependencies": {
     "@types/jest": "^29.5.14",
-    "@types/node": "^24.0.4",
-    "@typescript-eslint/eslint-plugin": "^8.35.0",
-    "@typescript-eslint/parser": "^8.35.0",
+    "@types/node": "^24.0.11",
+    "@typescript-eslint/eslint-plugin": "^8.36.0",
+    "@typescript-eslint/parser": "^8.36.0",
     "aws-sdk-client-mock": "^4.1.0",
-    "eslint": "^9.29.0",
+    "eslint": "^9.30.1",
     "eslint-plugin-import": "^2.32.0",
-    "eslint-plugin-jest": "^20.0.1",
-    "eslint-plugin-n": "^17.20.0",
+    "eslint-plugin-jest": "^29.0.1",
+    "eslint-plugin-n": "^17.21.0",
     "eslint-plugin-node": "^11.1.0",
     "eslint-plugin-promise": "^7.2.1",
     "github-changes": "^2.0.3",
     "jest": "^29.7.0",
     "jest-mock": "^30.0.2",
-    "prettier": "^3.6.1",
+    "prettier": "^3.6.2",
     "ts-jest": "^29.4.0",
     "tsc-watch": "^7.1.1",
     "typescript": "^5.8.3"
@@ -63,7 +63,7 @@
     "lib/**/*"
   ],
   "dependencies": {
-    "aws-cdk-lib": "^2.202.0",
+    "aws-cdk-lib": "^2.206.0",
     "constructs": "^10.4.2"
   },
   "workspaces": [