Merge branch 'dl-entrypoint' of https://github.com/firebase/firebase-functions into dl-entrypoint

Author: taeold

CHANGELOG.md

@@ -1 +1,4 @@
-- Future Extensions support (#1590)
+- Fix retry in event triggered functions. (#1463)
+- Expose retry configuration in v2 RTDB trigger (#1588)
+- Fix CORS options for v2 callable functions (#1564)
+- Remove invalid `enforceAppCheck` option for v2 onRequest trigger (#1477)

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "firebase-functions",
-  "version": "5.0.1",
+  "version": "5.1.0",
   "description": "Firebase SDK for Cloud Functions",
   "keywords": [
     "firebase",

package.json

@@ -1,4 +1,4 @@
-FROM node:14
+FROM node:20
 
 # Install dependencies
 RUN apt-get update && \

scripts/publish-container/Dockerfile

@@ -280,6 +280,40 @@ describe("database", () => {
         },
       });
     });
+
+    it("should supply retry", () => {
+      const func = database.onChangedOperation(
+        database.writtenEventType,
+        {
+          ref: "/foo/{path=**}/{bar}/",
+          instance: "my-instance",
+          region: "us-central1",
+          cpu: "gcf_gen1",
+          minInstances: 2,
+          retry: true,
+        },
+        () => 2
+      );
+
+      expect(func.__endpoint).to.deep.equal({
+        ...MINIMAL_V2_ENDPOINT,
+        platform: "gcfv2",
+        cpu: "gcf_gen1",
+        minInstances: 2,
+        region: ["us-central1"],
+        labels: {},
+        eventTrigger: {
+          eventType: database.writtenEventType,
+          eventFilters: {
+            instance: "my-instance",
+          },
+          eventFilterPathPatterns: {
+            ref: "foo/{path=**}/{bar}",
+          },
+          retry: true,
+        },
+      });
+    });
   });
 
   describe("onOperation", () => {

spec/v2/providers/database.spec.ts

@@ -87,6 +87,13 @@ function removeCircular(obj: any, refs: any[] = []): any {
  * @public
  */
 export function write(entry: LogEntry) {
+  const ctx = traceContext.getStore();
+  if (ctx?.traceId) {
+    entry[
+      "logging.googleapis.com/trace"
+    ] = `projects/${process.env.GCLOUD_PROJECT}/traces/${ctx.traceId}`;
+  }
+
   UNPATCHED_CONSOLE[CONSOLE_SEVERITY[entry.severity]](JSON.stringify(removeCircular(entry)));
 }
 
@@ -147,17 +154,13 @@ function entryFromArgs(severity: LogSeverity, args: any[]): LogEntry {
   if (lastArg && typeof lastArg === "object" && lastArg.constructor === Object) {
     entry = args.pop();
   }
-  const ctx = traceContext.getStore();
 
   // mimic `console.*` behavior, see https://nodejs.org/api/console.html#console_console_log_data_args
   let message = format(...args);
   if (severity === "ERROR" && !args.find((arg) => arg instanceof Error)) {
     message = new Error(message).stack || message;
   }
   const out: LogEntry = {
-    "logging.googleapis.com/trace": ctx?.traceId
-      ? `projects/${process.env.GCLOUD_PROJECT}/traces/${ctx.traceId}`
-      : undefined,
     ...entry,
     severity,
   };

src/logger/index.ts

@@ -111,11 +111,10 @@ export const storageBucket: Param<string> = new InternalExpression(
 
 /**
  * Declares a secret param, that will persist values only in Cloud Secret Manager.
- * Secrets are stored interally as bytestrings. Use `ParamOptions.as` to provide type
+ * Secrets are stored internally as bytestrings. Use `ParamOptions.as` to provide type
  * hinting during parameter resolution.
  *
  * @param name The name of the environment variable to use to load the parameter.
- * @param options Configuration options for the parameter.
  * @returns A parameter with a `string` return type for `.value`.
  */
 export function defineSecret(name: string): SecretParam {

src/params/index.ts

@@ -175,7 +175,7 @@ type ParamValueType = "string" | "list" | "boolean" | "int" | "float" | "secret"
 /** Create a select input from a series of values. */
 export function select<T>(options: T[]): SelectInput<T>;
 
-/** Create a select input from a map of labels to vaues. */
+/** Create a select input from a map of labels to values. */
 export function select<T>(optionsWithLabels: Record<string, T>): SelectInput<T>;
 
 /** Create a select input from a series of values or a map of labels to values */

src/params/types.ts

@@ -37,7 +37,7 @@ export function resetCache(): void {
  * Store and retrieve project configuration data such as third-party API
  * keys or other settings. You can set configuration values using the
  * Firebase CLI as described in
- * [Environment Configuration](/docs/functions/config-env).
+ * https://firebase.google.com/docs/functions/config-env.
  */
 export function config(): Record<string, any> {
   // K_CONFIGURATION is only set in GCFv2

src/v1/config.ts

@@ -135,7 +135,7 @@ export interface GlobalOptions {
    * The minimum timeout for a 2nd gen function is 1s. The maximum timeout for a
    * function depends on the type of function: Event handling functions have a
    * maximum timeout of 540s (9 minutes). HTTPS and callable functions have a
-   * maximum timeout of 36,00s (1 hour). Task queue functions have a maximum
+   * maximum timeout of 3,600s (1 hour). Task queue functions have a maximum
    * timeout of 1,800s (30 minutes).
    */
   timeoutSeconds?: number | Expression<number> | ResetValue;