sst 2.5.1 → 2.5.3

package/bootstrap.js

@@ -146,7 +146,7 @@ export async function bootstrapSST() {
     const fn = new Function(stack, "MetadataHandler", {
         code: Code.fromAsset(path.resolve(__dirname, "support/bootstrap-metadata-function")),
         handler: "index.handler",
-        runtime: region?.startsWith("us-gov-") || region?.startsWith("cn-")
+        runtime: region?.startsWith("us-gov-")
             ? Runtime.NODEJS_16_X
             : Runtime.NODEJS_18_X,
         environment: {

package/cli/commands/dev.js

@@ -69,7 +69,7 @@ export const dev = (program) => program.command(["dev", "start"], "Work on your
         });
         bus.subscribe("function.build.success", async (evt) => {
             const info = useFunctions().fromID(evt.properties.functionID);
-            if (!info.enableLiveDev)
+            if (info.enableLiveDev === false)
                 return;
             Colors.line(Colors.dim(Colors.prefix, "Built", info.handler));
         });

package/cli/telemetry/post-payload.js

@@ -16,8 +16,8 @@ export function postPayload(endpoint, body) {
             req.write(JSON.stringify(body));
             req.end();
         }
-        catch {
-            resolve();
+        catch (ex) {
+            reject(ex);
         }
     });
 }

package/cli/telemetry/telemetry.js

@@ -56,7 +56,7 @@ function notify() {
 function willNotRecord() {
     return !isEnabled() || !!process.env.SST_TELEMETRY_DISABLED;
 }
-function record(name, properties) {
+async function record(name, properties) {
     if (willNotRecord()) {
         return Promise.resolve();
     }
@@ -65,16 +65,19 @@ function record(name, properties) {
         projectId,
         sessionId,
     };
-    return postPayload(TELEMETRY_API, {
-        context,
-        environment: getEnvironmentData(),
-        events: [
-            {
-                name,
-                properties,
-            },
-        ],
-    });
+    try {
+        await postPayload(TELEMETRY_API, {
+            context,
+            environment: getEnvironmentData(),
+            events: [
+                {
+                    name,
+                    properties,
+                },
+            ],
+        });
+    }
+    catch { }
 }
 function getAnonymousId() {
     const val = conf && conf.get(TELEMETRY_KEY_ID);

package/constructs/App.d.ts

@@ -130,7 +130,7 @@ export declare class App extends CDKApp {
      *
      * @example
      * ```js
-     * app.addDefaultFunctionPermissions({
+     * app.addDefaultFunctionEnv({
      *   "MY_ENV_VAR": "my-value"
      * })
      * ```

package/constructs/App.js

@@ -145,7 +145,7 @@ export class App extends CDKApp {
      *
      * @example
      * ```js
-     * app.addDefaultFunctionPermissions({
+     * app.addDefaultFunctionEnv({
      *   "MY_ENV_VAR": "my-value"
      * })
      * ```

package/constructs/Function.js

@@ -20,6 +20,7 @@ import { RetentionDays } from "aws-cdk-lib/aws-logs";
 import { Token, Size as CDKSize, Duration as CDKDuration } from "aws-cdk-lib";
 import { Effect, PolicyStatement } from "aws-cdk-lib/aws-iam";
 import { StringParameter } from "aws-cdk-lib/aws-ssm";
+import { useBootstrap } from "../bootstrap.js";
 const __dirname = url.fileURLToPath(new URL(".", import.meta.url));
 const supportedRuntimes = {
     rust: CDKRuntime.PROVIDED_AL2,
@@ -159,13 +160,21 @@ export class Function extends CDKFunction {
                 ...(debugOverrideProps || {}),
             });
             this.addEnvironment("SST_FUNCTION_ID", this.node.addr);
-            this.attachPermissions([
-                new PolicyStatement({
-                    actions: ["iot:*", "s3:*"],
-                    effect: Effect.ALLOW,
-                    resources: ["*"],
-                }),
-            ]);
+            useDeferredTasks().add(async () => {
+                const bootstrap = await useBootstrap();
+                this.attachPermissions([
+                    new PolicyStatement({
+                        actions: ["iot:*"],
+                        effect: Effect.ALLOW,
+                        resources: ["*"],
+                    }),
+                    new PolicyStatement({
+                        actions: ["s3:*"],
+                        effect: Effect.ALLOW,
+                        resources: [`arn:aws:s3:::${bootstrap.bucket}`],
+                    }),
+                ]);
+            });
         }
         // Handle build
         else {

package/constructs/NextjsSite.js

@@ -26,7 +26,7 @@ const __dirname = url.fileURLToPath(new URL(".", import.meta.url));
 export class NextjsSite extends SsrSite {
     constructor(scope, id, props) {
         super(scope, id, {
-            buildCommand: "npx --yes open-next@^0.8.0 build",
+            buildCommand: "npx --yes open-next@^0.9.0 build",
             ...props,
         });
     }
@@ -263,7 +263,7 @@ export class NextjsSite extends SsrSite {
         const fallbackOriginGroup = new OriginGroup({
             primaryOrigin: serverOrigin,
             fallbackOrigin: s3Origin,
-            fallbackStatusCodes: [404],
+            fallbackStatusCodes: [503],
         });
         return {
             origin: fallbackOriginGroup,

package/constructs/Topic.d.ts

@@ -160,7 +160,7 @@ export declare class Topic extends Construct implements SSTConstruct {
      * Add subscribers to the topic.
      *
      * @example
-     * ```js {5}
+     * ```js
      * const topic = new Topic(stack, "Topic", {
      *   subscribers: {
      *     subscriber1: "src/function1.handler",
@@ -194,7 +194,7 @@ export declare class Topic extends Construct implements SSTConstruct {
     /**
      * Binds the given list of resources to a specific subscriber.
      * @example
-     * ```js {5}
+     * ```js {8}
      * const topic = new Topic(stack, "Topic", {
      *   subscribers: {
      *     subscriber1: "src/function1.handler",
@@ -225,7 +225,7 @@ export declare class Topic extends Construct implements SSTConstruct {
     /**
      * Attaches the list of permissions to a specific subscriber.
      * @example
-     * ```js {5}
+     * ```js {8}
      * const topic = new Topic(stack, "Topic", {
      *   subscribers: {
      *     subscriber1: "src/function1.handler",

package/constructs/Topic.js

@@ -80,7 +80,7 @@ export class Topic extends Construct {
      * Add subscribers to the topic.
      *
      * @example
-     * ```js {5}
+     * ```js
      * const topic = new Topic(stack, "Topic", {
      *   subscribers: {
      *     subscriber1: "src/function1.handler",
@@ -121,7 +121,7 @@ export class Topic extends Construct {
     /**
      * Binds the given list of resources to a specific subscriber.
      * @example
-     * ```js {5}
+     * ```js {8}
      * const topic = new Topic(stack, "Topic", {
      *   subscribers: {
      *     subscriber1: "src/function1.handler",
@@ -163,7 +163,7 @@ export class Topic extends Construct {
     /**
      * Attaches the list of permissions to a specific subscriber.
      * @example
-     * ```js {5}
+     * ```js {8}
      * const topic = new Topic(stack, "Topic", {
      *   subscribers: {
      *     subscriber1: "src/function1.handler",

package/iot.js

@@ -52,7 +52,7 @@ export const useIOT = Context.memo(async () => {
                 },
             ];
         }
-        const parts = json.match(/.{1,100000}/g);
+        const parts = json.match(/.{1,50000}/g);
         if (!parts)
             return [];
         Logger.debug("Encoded iot message into", parts?.length, "parts");

package/node/future/auth/adapter/github.d.ts

@@ -1,5 +1,11 @@
 import { OauthBasicConfig } from "./oauth.js";
-export declare const GithubAdapter: (config: OauthBasicConfig) => () => Promise<{
+import { OidcBasicConfig } from "./oidc.js";
+type Config = ({
+    mode: "oauth";
+} & OauthBasicConfig) | ({
+    mode: "oidc";
+} & OidcBasicConfig);
+export declare const GithubAdapter: (config: Config) => () => Promise<{
     type: "success";
     properties: {
         tokenset: import("openid-client").TokenSet;
@@ -14,3 +20,4 @@ export declare const GithubAdapter: (config: OauthBasicConfig) => () => Promise<
         };
     };
 }>;
+export {};

package/node/future/auth/adapter/github.js

@@ -9,7 +9,7 @@ const issuer = new Issuer({
 export const GithubAdapter = 
 /* @__PURE__ */
 (config) => {
-    if (config.clientSecret) {
+    if (config.mode === "oauth") {
         return OauthAdapter({
             issuer,
             ...config,
@@ -17,6 +17,7 @@ export const GithubAdapter =
     }
     return OidcAdapter({
         issuer,
+        scope: "openid email profile",
         ...config,
     });
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sst",
-  "version": "2.5.1",
+  "version": "2.5.3",
   "bin": {
     "sst": "cli/sst.js"
   },

package/project.js

@@ -82,6 +82,16 @@ export async function initProject(globals) {
             artifacts: path.join(out, "artifacts"),
         },
     };
+    // Cleanup old config files
+    (async function () {
+        const files = await fs.readdir(project.paths.root);
+        for (const file of files) {
+            if (file.startsWith(".sst.config")) {
+                await fs.unlink(path.join(project.paths.root, file));
+                Logger.debug(`Removed old config file ${file}`);
+            }
+        }
+    })();
     ProjectContext.provide(project);
     dotenv.config({
         path: path.join(project.paths.root, `.env.${project.config.stage}`),

package/sst.mjs

@@ -387,6 +387,15 @@ async function initProject(globals) {
       artifacts: path4.join(out, "artifacts")
     }
   };
+  (async function() {
+    const files = await fs4.readdir(project.paths.root);
+    for (const file2 of files) {
+      if (file2.startsWith(".sst.config")) {
+        await fs4.unlink(path4.join(project.paths.root, file2));
+        Logger.debug(`Removed old config file ${file2}`);
+      }
+    }
+  })();
   ProjectContext.provide(project);
   dotenv.config({
     path: path4.join(project.paths.root, `.env.${project.config.stage}`),
@@ -488,8 +497,8 @@ function postPayload(endpoint, body) {
       );
       req.write(JSON.stringify(body));
       req.end();
-    } catch {
-      resolve();
+    } catch (ex) {
+      reject(ex);
     }
   });
 }
@@ -636,7 +645,7 @@ function notify() {
 function willNotRecord() {
   return !isEnabled() || !!process.env.SST_TELEMETRY_DISABLED;
 }
-function record(name, properties) {
+async function record(name, properties) {
   if (willNotRecord()) {
     return Promise.resolve();
   }
@@ -645,16 +654,19 @@ function record(name, properties) {
     projectId,
     sessionId
   };
-  return postPayload(TELEMETRY_API, {
-    context,
-    environment: getEnvironmentData(),
-    events: [
-      {
-        name,
-        properties
-      }
-    ]
-  });
+  try {
+    await postPayload(TELEMETRY_API, {
+      context,
+      environment: getEnvironmentData(),
+      events: [
+        {
+          name,
+          properties
+        }
+      ]
+    });
+  } catch {
+  }
 }
 function getAnonymousId() {
   const val = conf && conf.get(TELEMETRY_KEY_ID);
@@ -5211,7 +5223,7 @@ async function bootstrapSST() {
       path15.resolve(__dirname2, "support/bootstrap-metadata-function")
     ),
     handler: "index.handler",
-    runtime: region?.startsWith("us-gov-") || region?.startsWith("cn-") ? Runtime2.NODEJS_16_X : Runtime2.NODEJS_18_X,
+    runtime: region?.startsWith("us-gov-") ? Runtime2.NODEJS_16_X : Runtime2.NODEJS_18_X,
     environment: {
       BUCKET_NAME: bucket.bucketName
     },
@@ -5777,7 +5789,7 @@ var init_iot = __esm({
             }
           ];
         }
-        const parts = json.match(/.{1,100000}/g);
+        const parts = json.match(/.{1,50000}/g);
         if (!parts)
           return [];
         Logger.debug("Encoded iot message into", parts?.length, "parts");
@@ -6851,7 +6863,7 @@ var dev = (program2) => program2.command(
       });
       bus.subscribe("function.build.success", async (evt) => {
         const info = useFunctions3().fromID(evt.properties.functionID);
-        if (!info.enableLiveDev)
+        if (info.enableLiveDev === false)
           return;
         Colors2.line(Colors2.dim(Colors2.prefix, "Built", info.handler));
       });