sst 2.35.1 → 2.36.0

package/constructs/NextjsSite.d.ts

@@ -21,10 +21,10 @@ export interface NextjsSiteProps extends Omit<SsrSiteProps, "nodejs"> {
      * How the logs are stored in CloudWatch
      * - "combined" - Logs from all routes are stored in the same log group.
      * - "per-route" - Logs from each route are stored in a separate log group.
-     * @default "combined"
+     * @default "per-route"
      * @example
      * ```js
-     * logging: "per-route",
+     * logging: "combined",
      * ```
      */
     logging?: "combined" | "per-route";
@@ -134,7 +134,7 @@ export declare class NextjsSite extends SsrSite {
     private appPathsManifest?;
     private pagesManifest?;
     private prerenderManifest?;
-    constructor(scope: Construct, id: string, props?: NextjsSiteProps);
+    constructor(scope: Construct, id: string, rawProps?: NextjsSiteProps);
     static buildDefaultServerCachePolicyProps(): CachePolicyProps;
     protected plan(bucket: Bucket): {
         cloudFrontFunctions?: {
@@ -205,10 +205,10 @@ export declare class NextjsSite extends SsrSite {
                      * How the logs are stored in CloudWatch
                      * - "combined" - Logs from all routes are stored in the same log group.
                      * - "per-route" - Logs from each route are stored in a separate log group.
-                     * @default "combined"
+                     * @default "per-route"
                      * @example
                      * ```js
-                     * logging: "per-route",
+                     * logging: "combined",
                      * ```
                      */
                     environment?: Record<string, string> | undefined;
@@ -312,10 +312,10 @@ export declare class NextjsSite extends SsrSite {
                      * How the logs are stored in CloudWatch
                      * - "combined" - Logs from all routes are stored in the same log group.
                      * - "per-route" - Logs from each route are stored in a separate log group.
-                     * @default "combined"
+                     * @default "per-route"
                      * @example
                      * ```js
-                     * logging: "per-route",
+                     * logging: "combined",
                      * ```
                      */
                     environment?: Record<string, string> | undefined;

package/constructs/NextjsSite.js

@@ -20,7 +20,7 @@ import { useFunctions } from "./Function.js";
 import { useDeferredTasks } from "./deferred_task.js";
 import { Logger } from "../logger.js";
 const LAYER_VERSION = "2";
-const DEFAULT_OPEN_NEXT_VERSION = "2.3.0";
+const DEFAULT_OPEN_NEXT_VERSION = "2.3.1";
 const DEFAULT_CACHE_POLICY_ALLOWED_HEADERS = [
     "accept",
     "rsc",
@@ -46,21 +46,28 @@ export class NextjsSite extends SsrSite {
     appPathsManifest;
     pagesManifest;
     prerenderManifest;
-    constructor(scope, id, props) {
-        const streaming = props?.experimental?.streaming ?? false;
-        const disableDynamoDBCache = props?.experimental?.disableDynamoDBCache ?? false;
-        const disableIncrementalCache = props?.experimental?.disableIncrementalCache ?? false;
+    constructor(scope, id, rawProps) {
+        const props = {
+            logging: rawProps?.logging ?? "per-route",
+            experimental: {
+                streaming: rawProps?.experimental?.streaming ?? false,
+                disableDynamoDBCache: rawProps?.experimental?.disableDynamoDBCache ?? false,
+                disableIncrementalCache: rawProps?.experimental?.disableIncrementalCache ?? false,
+                ...rawProps?.experimental,
+            },
+            ...rawProps,
+        };
         super(scope, id, {
             buildCommand: [
                 "npx",
                 "--yes",
                 `open-next@${props?.openNextVersion ?? DEFAULT_OPEN_NEXT_VERSION}`,
                 "build",
-                ...(streaming ? ["--streaming"] : []),
-                ...(disableDynamoDBCache
+                ...(props.experimental.streaming ? ["--streaming"] : []),
+                ...(props.experimental.disableDynamoDBCache
                     ? ["--dangerously-disable-dynamodb-cache"]
                     : []),
-                ...(disableIncrementalCache
+                ...(props.experimental.disableIncrementalCache
                     ? ["--dangerously-disable-incremental-cache"]
                     : []),
             ].join(" "),
@@ -70,9 +77,9 @@ export class NextjsSite extends SsrSite {
             this.disableDefaultLogging();
             this.uploadSourcemaps();
         }
-        if (!disableIncrementalCache) {
+        if (!props.experimental.disableIncrementalCache) {
             this.createRevalidationQueue();
-            if (!disableDynamoDBCache) {
+            if (!props.experimental.disableDynamoDBCache) {
                 this.createRevalidationTable();
             }
         }
@@ -564,8 +571,6 @@ export class NextjsSite extends SsrSite {
             this.props.logging === "per-route");
     }
     disableDefaultLogging() {
-        // Note: keep default logs enabled
-        return;
         const stack = Stack.of(this);
         const server = this.serverFunction;
         const policy = new Policy(this, "DisableLoggingPolicy", {

package/package.json

@@ -1,7 +1,7 @@
 {
   "sideEffects": false,
   "name": "sst",
-  "version": "2.35.1",
+  "version": "2.36.0",
   "bin": {
     "sst": "cli/sst.js"
   },
@@ -120,7 +120,7 @@
     "@types/ws": "^8.5.3",
     "@types/yargs": "^17.0.13",
     "archiver": "^5.3.1",
-    "astro-sst": "2.35.1",
+    "astro-sst": "2.36.0",
     "async": "^3.2.4",
     "tsx": "^3.12.1",
     "typescript": "^5.2.2",