firebase-functions icon indicating copy to clipboard operation
firebase-functions copied to clipboard

Firebase Crashlytics - Cannot use dynamic params for declaring appId

Open BenjaminBihr opened this issue 1 year ago • 2 comments

Related issues

I couldn't find any related issues.

Version info

node: 20

firebase-functions: 5.0.0

firebase-tools: 13.15.2

Test case


import {defineString} from "firebase-functions/params";
import {onVelocityAlertPublished} from "firebase-functions/v2/alerts/crashlytics";
import {logger} from "firebase-functions";

const APP_ID = defineString('APP_ID');

export const detectVelocityAlert = onVelocityAlertPublished({
  appId: APP_ID,
}, async (event) => {
  logger.log("New velocity alert spotted!");
});

Steps to reproduce

  1. Create a Cloud Function using onVelocityAlertPublished.
  2. Define the appId using defineString('APP_ID') from firebase-functions/params.
  3. Attempt to deploy or run the function.

Expected behavior

I expect the function to accept a parameter defined via defineString('APP_ID') for the appId in the onVelocityAlertPublished function configuration.

Actual behavior

When I try to use defineString('APP_ID') to set the appId, the function fails to deploy because it appears to only accept a string literal, not a StringParams.

Were you able to successfully deploy your functions?

No, the deployment fails.

BenjaminBihr avatar Aug 20 '24 15:08 BenjaminBihr

I couldn't figure out how to label this issue, so I've labeled it for a human to triage. Hang tight.

google-oss-bot avatar Aug 20 '24 15:08 google-oss-bot

Hello @BenjaminBihr. Thanks for bringing this up. Seems a valid feature request

exaby73 avatar Aug 29 '24 16:08 exaby73