sst 2.24.22 → 2.24.23

package/constructs/NextjsSite.d.ts

@@ -63,7 +63,7 @@ export declare class NextjsSite extends SsrSite {
     private createWarmer;
     protected createCloudFrontDistributionForRegional(): Distribution;
     protected createCloudFrontDistributionForEdge(): Distribution;
-    protected buildServerCachePolicy(): CachePolicy;
+    protected useServerBehaviorCachePolicy(): CachePolicy;
     private buildImageBehavior;
     protected generateBuildId(): string;
     getConstructMetadata(): {

package/constructs/NextjsSite.js

@@ -255,8 +255,7 @@ export class NextjsSite extends SsrSite {
          */
         const { customDomain, cdk } = this.props;
         const cfDistributionProps = cdk?.distribution || {};
-        const cachePolicy = cdk?.serverCachePolicy ?? this.buildServerCachePolicy();
-        const serverBehavior = this.buildDefaultBehaviorForRegional(cachePolicy);
+        const serverBehavior = this.buildDefaultBehaviorForRegional();
         return new Distribution(this, "CDN", {
             scopeOverride: this,
             customDomain,
@@ -271,7 +270,7 @@ export class NextjsSite extends SsrSite {
                     additionalBehaviors: {
                         "api/*": serverBehavior,
                         "_next/data/*": serverBehavior,
-                        "_next/image*": this.buildImageBehavior(cachePolicy),
+                        "_next/image*": this.buildImageBehavior(),
                         ...(cfDistributionProps.additionalBehaviors || {}),
                     },
                 },
@@ -281,8 +280,7 @@ export class NextjsSite extends SsrSite {
     createCloudFrontDistributionForEdge() {
         const { customDomain, cdk } = this.props;
         const cfDistributionProps = cdk?.distribution || {};
-        const cachePolicy = cdk?.serverCachePolicy ?? this.buildServerCachePolicy();
-        const serverBehavior = this.buildDefaultBehaviorForEdge(cachePolicy);
+        const serverBehavior = this.buildDefaultBehaviorForEdge();
         return new Distribution(this, "CDN", {
             scopeOverride: this,
             customDomain,
@@ -297,15 +295,15 @@ export class NextjsSite extends SsrSite {
                     additionalBehaviors: {
                         "api/*": serverBehavior,
                         "_next/data/*": serverBehavior,
-                        "_next/image*": this.buildImageBehavior(cachePolicy),
+                        "_next/image*": this.buildImageBehavior(),
                         ...(cfDistributionProps.additionalBehaviors || {}),
                     },
                 },
             },
         });
     }
-    buildServerCachePolicy() {
-        return super.buildServerCachePolicy([
+    useServerBehaviorCachePolicy() {
+        return super.useServerBehaviorCachePolicy([
             "accept",
             "rsc",
             "next-router-prefetch",
@@ -313,7 +311,7 @@ export class NextjsSite extends SsrSite {
             "next-url",
         ]);
     }
-    buildImageBehavior(cachePolicy) {
+    buildImageBehavior() {
         const { cdk, regional } = this.props;
         const imageFn = this.createImageOptimizationFunction();
         const imageFnUrl = imageFn.addFunctionUrl({
@@ -327,7 +325,7 @@ export class NextjsSite extends SsrSite {
             allowedMethods: AllowedMethods.ALLOW_ALL,
             cachedMethods: CachedMethods.CACHE_GET_HEAD_OPTIONS,
             compress: true,
-            cachePolicy,
+            cachePolicy: cdk?.serverCachePolicy ?? this.useServerBehaviorCachePolicy(),
             responseHeadersPolicy: cdk?.responseHeadersPolicy,
             edgeLambdas: regional?.enableServerUrlIamAuth
                 ? [

package/constructs/RemixSite.d.ts

@@ -20,6 +20,7 @@ export declare class RemixSite extends SsrSite {
         serverBuildOutputFile: string;
         clientBuildOutputDir: string;
         clientBuildVersionedSubDir: string;
+        clientCFFunctionInjection: string;
     };
     private createServerLambdaBundle;
     protected createFunctionForRegional(): SsrFunction;

package/constructs/RemixSite.js

@@ -53,6 +53,11 @@ export class RemixSite extends SsrSite {
             serverBuildOutputFile: "build/index.js",
             clientBuildOutputDir: "public",
             clientBuildVersionedSubDir: "build",
+            // Note: When using libraries like remix-flat-routes the file can
+            // contains special characters like "+". It needs to be encoded.
+            clientCFFunctionInjection: `
+       request.uri = request.uri.split('/').map(encodeURIComponent).join('/');
+      `,
         };
     }
     createServerLambdaBundle(wrapperFile) {

package/constructs/Service.d.ts

@@ -6,6 +6,7 @@ import { FunctionBindingProps } from "./util/functionBinding.js";
 import { IVpc } from "aws-cdk-lib/aws-ec2";
 import { Cluster, ContainerDefinitionOptions } from "aws-cdk-lib/aws-ecs";
 import { RetentionDays } from "aws-cdk-lib/aws-logs";
+import { ApplicationTargetGroupProps } from "aws-cdk-lib/aws-elasticloadbalancingv2";
 declare const supportedCpus: {
     "0.25 vCPU": number;
     "0.5 vCPU": number;
@@ -248,7 +249,9 @@ export interface ServiceProps {
          * @example
          * ```js
          * {
-         *   cloudfrontDistribution: false
+         *   cdk: {
+         *     cloudfrontDistribution: false
+         *   }
          * }
          * ```
          */
@@ -259,11 +262,30 @@ export interface ServiceProps {
          * @example
          * ```js
          * {
-         *   applicationLoadBalancer: false
+         *   cdk: {
+         *     applicationLoadBalancer: false
+         *   }
          * }
          * ```
          */
         applicationLoadBalancer?: boolean;
+        /**
+         * Customize the Application Load Balancer's target group.
+         * @default true
+         * @example
+         * ```js
+         * {
+         *   cdk: {
+         *     applicationLoadBalancerTargetGroup: {
+         *       healthCheck: {
+         *         path: "/health"
+         *       }
+         *     }
+         *   }
+         * }
+         * ```
+         */
+        applicationLoadBalancerTargetGroup?: ApplicationTargetGroupProps;
         /**
          * Customizing the container definition for the ECS task.
          * @example

package/constructs/Service.js

@@ -427,6 +427,8 @@ export class Service extends Construct {
         const { cdk } = this.props;
         // Do not create load balancer if disabled
         if (cdk?.applicationLoadBalancer === false) {
+            if (cdk?.applicationLoadBalancerTargetGroup)
+                throw new VisibleError(`In the "${this.node.id}" Service, the "cdk.applicationLoadBalancerTargetGroup" cannot be applied if the Application Load Balancer is diabled.`);
             return {};
         }
         const alb = new ApplicationLoadBalancer(this, "LoadBalancer", {
@@ -437,6 +439,7 @@ export class Service extends Construct {
         const target = listener.addTargets("TargetGroup", {
             port: 80,
             targets: [service],
+            ...cdk?.applicationLoadBalancerTargetGroup,
         });
         return { alb, target };
     }

package/constructs/SsrSite.d.ts

@@ -19,6 +19,7 @@ export type SsrBuildConfig = {
     clientBuildOutputDir: string;
     clientBuildVersionedSubDir: string;
     clientBuildS3KeyPrefix?: string;
+    clientCFFunctionInjection?: string;
     prerenderedBuildOutputDir?: string;
     prerenderedBuildS3KeyPrefix?: string;
 };
@@ -284,7 +285,10 @@ export declare abstract class SsrSite extends Construct implements SSTConstruct
     private serverLambdaForDev?;
     private serverUrlSigningFunction?;
     protected bucket: Bucket;
-    private cfFunction;
+    private serverCfFunction?;
+    private serverBehaviorCachePolicy?;
+    private serverBehaviorOriginRequestPolicy?;
+    private staticCfFunction?;
     private s3Origin;
     private distribution;
     constructor(scope: Construct, id: string, props?: SsrSiteProps);
@@ -347,19 +351,22 @@ export declare abstract class SsrSite extends Construct implements SSTConstruct
     private grantServerS3Permissions;
     private grantServerCloudFrontPermissions;
     private createCloudFrontS3Origin;
-    private createCloudFrontFunction;
     protected createCloudFrontDistributionForRegional(): Distribution;
     protected createCloudFrontDistributionForEdge(): Distribution;
-    protected buildDefaultBehaviorForRegional(cachePolicy: ICachePolicy): BehaviorOptions;
-    protected useServerUrlSigningFunction(): EdgeFunction;
-    protected buildDefaultBehaviorForEdge(cachePolicy: ICachePolicy): BehaviorOptions;
-    protected buildBehaviorFunctionAssociations(): {
+    protected buildDefaultBehaviorForRegional(): BehaviorOptions;
+    protected buildDefaultBehaviorForEdge(): BehaviorOptions;
+    protected addStaticFileBehaviors(): void;
+    protected useServerBehaviorFunctionAssociations(): {
         eventType: CfFunctionEventType;
         function: CfFunction;
     }[];
-    protected addStaticFileBehaviors(): void;
-    protected buildServerCachePolicy(allowedHeaders?: string[]): CachePolicy;
-    protected buildServerOriginRequestPolicy(): import("aws-cdk-lib/aws-cloudfront").IOriginRequestPolicy;
+    protected useStaticBehaviorFunctionAssociations(): {
+        eventType: CfFunctionEventType;
+        function: CfFunction;
+    }[];
+    protected useServerUrlSigningFunction(): EdgeFunction;
+    protected useServerBehaviorCachePolicy(allowedHeaders?: string[]): CachePolicy;
+    private useServerBehaviorOriginRequestPolicy;
     private getS3ContentReplaceValues;
     private validateSiteExists;
     private validateTimeout;

package/constructs/SsrSite.js

@@ -51,7 +51,10 @@ export class SsrSite extends Construct {
     serverLambdaForDev;
     serverUrlSigningFunction;
     bucket;
-    cfFunction;
+    serverCfFunction;
+    serverBehaviorCachePolicy;
+    serverBehaviorOriginRequestPolicy;
+    staticCfFunction;
     s3Origin;
     distribution;
     constructor(scope, id, props) {
@@ -75,8 +78,8 @@ export class SsrSite extends Construct {
         this.writeTypesFile();
         useSites().add(stack.stackName, id, this.constructor.name, this.props);
         if (this.doNotDeploy) {
-            // @ts-ignore
-            this.cfFunction = this.bucket = this.s3Origin = this.distribution = null;
+            // @ts-expect-error
+            this.bucket = this.s3Origin = this.distribution = null;
             this.serverLambdaForDev = this.createFunctionForDev();
             return;
         }
@@ -92,7 +95,6 @@ export class SsrSite extends Construct {
         this.grantServerS3Permissions();
         // Create CloudFront
         this.s3Origin = this.createCloudFrontS3Origin();
-        this.cfFunction = this.createCloudFrontFunction();
         this.distribution = this.props.edge
             ? this.createCloudFrontDistributionForEdge()
             : this.createCloudFrontDistributionForRegional();
@@ -486,21 +488,9 @@ export class SsrSite extends Construct {
             originPath: "/" + (this.buildConfig.clientBuildS3KeyPrefix ?? ""),
         });
     }
-    createCloudFrontFunction() {
-        return new CfFunction(this, "CloudFrontFunction", {
-            code: CfFunctionCode.fromInline(`
-function handler(event) {
-  var request = event.request;
-  request.headers["x-forwarded-host"] = request.headers.host;
-  ${this.buildConfig.serverCFFunctionInjection || ""}
-  return request;
-}`),
-        });
-    }
     createCloudFrontDistributionForRegional() {
         const { customDomain, cdk } = this.props;
         const cfDistributionProps = cdk?.distribution || {};
-        const cachePolicy = cdk?.serverCachePolicy ?? this.buildServerCachePolicy();
         return new Distribution(this, "CDN", {
             scopeOverride: this,
             customDomain,
@@ -511,7 +501,7 @@ function handler(event) {
                     // Override props.
                     ...cfDistributionProps,
                     // these values can NOT be overwritten by cfDistributionProps
-                    defaultBehavior: this.buildDefaultBehaviorForRegional(cachePolicy),
+                    defaultBehavior: this.buildDefaultBehaviorForRegional(),
                     additionalBehaviors: {
                         ...(cfDistributionProps.additionalBehaviors || {}),
                     },
@@ -522,7 +512,6 @@ function handler(event) {
     createCloudFrontDistributionForEdge() {
         const { customDomain, cdk } = this.props;
         const cfDistributionProps = cdk?.distribution || {};
-        const cachePolicy = cdk?.serverCachePolicy ?? this.buildServerCachePolicy();
         return new Distribution(this, "CDN", {
             scopeOverride: this,
             customDomain,
@@ -533,7 +522,7 @@ function handler(event) {
                     // Override props.
                     ...cfDistributionProps,
                     // these values can NOT be overwritten by cfDistributionProps
-                    defaultBehavior: this.buildDefaultBehaviorForEdge(cachePolicy),
+                    defaultBehavior: this.buildDefaultBehaviorForEdge(),
                     additionalBehaviors: {
                         ...(cfDistributionProps.additionalBehaviors || {}),
                     },
@@ -541,7 +530,7 @@ function handler(event) {
             },
         });
     }
-    buildDefaultBehaviorForRegional(cachePolicy) {
+    buildDefaultBehaviorForRegional() {
         const { timeout, regional, cdk } = this.props;
         const cfDistributionProps = cdk?.distribution || {};
         const fnUrl = this.serverLambdaForRegional.addFunctionUrl({
@@ -562,12 +551,12 @@ function handler(event) {
             allowedMethods: AllowedMethods.ALLOW_ALL,
             cachedMethods: CachedMethods.CACHE_GET_HEAD_OPTIONS,
             compress: true,
-            cachePolicy,
+            cachePolicy: cdk?.serverCachePolicy ?? this.useServerBehaviorCachePolicy(),
             responseHeadersPolicy: cdk?.responseHeadersPolicy,
-            originRequestPolicy: this.buildServerOriginRequestPolicy(),
+            originRequestPolicy: this.useServerBehaviorOriginRequestPolicy(),
             ...(cfDistributionProps.defaultBehavior || {}),
             functionAssociations: [
-                ...this.buildBehaviorFunctionAssociations(),
+                ...this.useServerBehaviorFunctionAssociations(),
                 ...(cfDistributionProps.defaultBehavior?.functionAssociations || []),
             ],
             edgeLambdas: [
@@ -584,25 +573,7 @@ function handler(event) {
             ],
         };
     }
-    useServerUrlSigningFunction() {
-        this.serverUrlSigningFunction =
-            this.serverUrlSigningFunction ??
-                new EdgeFunction(this, "ServerUrlSigningFunction", {
-                    bundle: path.join(__dirname, "../support/signing-function"),
-                    runtime: "nodejs18.x",
-                    handler: "index.handler",
-                    timeout: 10,
-                    memorySize: 128,
-                    permissions: [
-                        new PolicyStatement({
-                            actions: ["lambda:InvokeFunctionUrl"],
-                            resources: [this.serverLambdaForRegional?.functionArn],
-                        }),
-                    ],
-                });
-        return this.serverUrlSigningFunction;
-    }
-    buildDefaultBehaviorForEdge(cachePolicy) {
+    buildDefaultBehaviorForEdge() {
         const { cdk } = this.props;
         const cfDistributionProps = cdk?.distribution || {};
         return {
@@ -611,12 +582,12 @@ function handler(event) {
             allowedMethods: AllowedMethods.ALLOW_ALL,
             cachedMethods: CachedMethods.CACHE_GET_HEAD_OPTIONS,
             compress: true,
-            cachePolicy,
+            cachePolicy: cdk?.serverCachePolicy ?? this.useServerBehaviorCachePolicy(),
             responseHeadersPolicy: cdk?.responseHeadersPolicy,
-            originRequestPolicy: this.buildServerOriginRequestPolicy(),
+            originRequestPolicy: this.useServerBehaviorOriginRequestPolicy(),
             ...(cfDistributionProps.defaultBehavior || {}),
             functionAssociations: [
-                ...this.buildBehaviorFunctionAssociations(),
+                ...this.useServerBehaviorFunctionAssociations(),
                 ...(cfDistributionProps.defaultBehavior?.functionAssociations || []),
             ],
             edgeLambdas: [
@@ -629,14 +600,6 @@ function handler(event) {
             ],
         };
     }
-    buildBehaviorFunctionAssociations() {
-        return [
-            {
-                eventType: CfFunctionEventType.VIEWER_REQUEST,
-                function: this.cfFunction,
-            },
-        ];
-    }
     addStaticFileBehaviors() {
         const { cdk } = this.props;
         // Create a template for statics behaviours
@@ -650,27 +613,93 @@ function handler(event) {
                 compress: true,
                 cachePolicy: CachePolicy.CACHING_OPTIMIZED,
                 responseHeadersPolicy: cdk?.responseHeadersPolicy,
+                functionAssociations: [
+                    ...this.useStaticBehaviorFunctionAssociations(),
+                ],
             });
         }
     }
-    buildServerCachePolicy(allowedHeaders) {
-        return new CachePolicy(this, "ServerCache", {
-            queryStringBehavior: CacheQueryStringBehavior.all(),
-            headerBehavior: allowedHeaders && allowedHeaders.length > 0
-                ? CacheHeaderBehavior.allowList(...allowedHeaders)
-                : CacheHeaderBehavior.none(),
-            cookieBehavior: CacheCookieBehavior.none(),
-            defaultTtl: CdkDuration.days(0),
-            maxTtl: CdkDuration.days(365),
-            minTtl: CdkDuration.days(0),
-            enableAcceptEncodingBrotli: true,
-            enableAcceptEncodingGzip: true,
-            comment: "SST server response cache policy",
-        });
+    useServerBehaviorFunctionAssociations() {
+        this.serverCfFunction =
+            this.serverCfFunction ??
+                new CfFunction(this, "CloudFrontFunction", {
+                    code: CfFunctionCode.fromInline(`
+function handler(event) {
+  var request = event.request;
+  request.headers["x-forwarded-host"] = request.headers.host;
+  ${this.buildConfig.serverCFFunctionInjection || ""}
+  return request;
+}`),
+                });
+        return [
+            {
+                eventType: CfFunctionEventType.VIEWER_REQUEST,
+                function: this.serverCfFunction,
+            },
+        ];
+    }
+    useStaticBehaviorFunctionAssociations() {
+        if (!this.buildConfig.clientCFFunctionInjection)
+            return [];
+        this.staticCfFunction =
+            this.staticCfFunction ??
+                new CfFunction(this, "CloudFrontFunctionForStaticBehavior", {
+                    code: CfFunctionCode.fromInline(`
+function handler(event) {
+  var request = event.request;
+  ${this.buildConfig.clientCFFunctionInjection || ""}
+  return request;
+}`),
+                });
+        return [
+            {
+                eventType: CfFunctionEventType.VIEWER_REQUEST,
+                function: this.staticCfFunction,
+            },
+        ];
+    }
+    useServerUrlSigningFunction() {
+        this.serverUrlSigningFunction =
+            this.serverUrlSigningFunction ??
+                new EdgeFunction(this, "ServerUrlSigningFunction", {
+                    bundle: path.join(__dirname, "../support/signing-function"),
+                    runtime: "nodejs18.x",
+                    handler: "index.handler",
+                    timeout: 10,
+                    memorySize: 128,
+                    permissions: [
+                        new PolicyStatement({
+                            actions: ["lambda:InvokeFunctionUrl"],
+                            resources: [this.serverLambdaForRegional?.functionArn],
+                        }),
+                    ],
+                });
+        return this.serverUrlSigningFunction;
+    }
+    useServerBehaviorCachePolicy(allowedHeaders) {
+        this.serverBehaviorCachePolicy =
+            this.serverBehaviorCachePolicy ??
+                new CachePolicy(this, "ServerCache", {
+                    queryStringBehavior: CacheQueryStringBehavior.all(),
+                    headerBehavior: allowedHeaders && allowedHeaders.length > 0
+                        ? CacheHeaderBehavior.allowList(...allowedHeaders)
+                        : CacheHeaderBehavior.none(),
+                    cookieBehavior: CacheCookieBehavior.none(),
+                    defaultTtl: CdkDuration.days(0),
+                    maxTtl: CdkDuration.days(365),
+                    minTtl: CdkDuration.days(0),
+                    enableAcceptEncodingBrotli: true,
+                    enableAcceptEncodingGzip: true,
+                    comment: "SST server response cache policy",
+                });
+        return this.serverBehaviorCachePolicy;
     }
-    buildServerOriginRequestPolicy() {
+    useServerBehaviorOriginRequestPolicy() {
         // CloudFront's Managed-AllViewerExceptHostHeader policy
-        return OriginRequestPolicy.fromOriginRequestPolicyId(this, "ServerOriginRequestPolicy", "b689b0a8-53d0-40ab-baf2-68738e2966ac");
+        this.serverBehaviorOriginRequestPolicy =
+            this.serverBehaviorOriginRequestPolicy ??
+                OriginRequestPolicy.fromOriginRequestPolicyId(this, "ServerOriginRequestPolicy", "b689b0a8-53d0-40ab-baf2-68738e2966ac");
+        return this.serverBehaviorOriginRequestPolicy;
     }
     /////////////////////
     // Helper Functions

package/package.json

@@ -1,7 +1,7 @@
 {
   "sideEffects": false,
   "name": "sst",
-  "version": "2.24.22",
+  "version": "2.24.23",
   "bin": {
     "sst": "cli/sst.js"
   },

package/runtime/handlers/java.js

@@ -68,26 +68,22 @@ export const useJavaHandler = Context.memo(async () => {
             const buildTask = input.props.java?.buildTask || "build";
             const outputDir = input.props.java?.buildOutputDir || "distributions";
             sources.set(input.functionID, srcPath);
-            async function build() {
-                // build
-                await execAsync(`${buildBinary} ${buildTask} -Dorg.gradle.logging.level=${process.env.DEBUG ? "debug" : "lifecycle"}`, {
-                    cwd: srcPath,
-                });
-                // unzip
-                const buildOutput = path.join(srcPath, "build", outputDir);
-                const zip = (await fs.readdir(buildOutput)).find((f) => f.endsWith(".zip"));
-                await new Promise((resolve, reject) => {
-                    const zipper = new AdmZip(path.join(buildOutput, zip));
-                    zipper.extractAllToAsync(input.out, false, false, (err) => err ? reject(err) : resolve(undefined));
-                });
-            }
             try {
-                // Run gradle build once per directory. Otherwise they'll interfere
+                // Build
+                // Note: run gradle build once per directory. Otherwise they'll interfere
                 // with one another
-                const buildPromise = runningBuilds.get(buildBinary) ?? build();
+                const buildPromise = runningBuilds.get(buildBinary) ??
+                    execAsync(`${buildBinary} ${buildTask} -Dorg.gradle.logging.level=${process.env.DEBUG ? "debug" : "lifecycle"}`, {
+                        cwd: srcPath,
+                    });
                 runningBuilds.set(buildBinary, buildPromise);
                 await buildPromise;
                 runningBuilds.delete(buildBinary);
+                // unzip
+                const buildOutput = path.join(srcPath, "build", outputDir);
+                const zip = (await fs.readdir(buildOutput)).find((f) => f.endsWith(".zip"));
+                const zipper = new AdmZip(path.join(buildOutput, zip));
+                zipper.extractAllTo(input.out, false, false);
                 return {
                     type: "success",
                     handler: input.props.handler,