Properly escape log URL and print it at the end for functions:log command

CHANGELOG.md

@@ -0,0 +1 @@
+- Added Cloud Logging URL at the end of `functions:log` command (#8211)

src/commands/functions-log.ts

@@ -1,8 +1,9 @@
 import * as opn from "open";
-import * as qs from "querystring";
+import { URLSearchParams } from "url";
 
 import { Command } from "../command";
 import { FirebaseError } from "../error";
+import { logger } from "../logger";
 import * as cloudlogging from "../gcp/cloudlogging";
 import * as functionsLog from "../functions/functionslog";
 import { needProjectId } from "../projectUtils";
@@ -17,26 +18,28 @@
   .option("-n, --lines <num_lines>", "specify number of log lines to fetch")
   .option("--open", "open logs page in web browser")
   .before(requirePermissions, ["logging.logEntries.list", "logging.logs.list"])
   .action(async (options: any) => {
     try {
       const projectId = needProjectId(options);
       const apiFilter = functionsLog.getApiFilter(options.only);
+      const filterParams = new URLSearchParams(apiFilter);
+      const url = `https://console.developers.google.com/logs/viewer?advancedFilter=${filterParams.toString()}&project=${projectId}`;
+
       if (options.open) {
-        const url = `https://console.developers.google.com/logs/viewer?advancedFilter=${qs.escape(
-          apiFilter,
-        )}&project=${projectId}`;
         opn(url);
         return;
       }
+
       const entries = await cloudlogging.listEntries(
         projectId,
         apiFilter,
         options.lines || 35,
         "desc",
       );
       functionsLog.logEntries(entries);
+      logger.info(`\nSee full logs at: ${url}`);
       return entries;
     } catch (err: any) {
       throw new FirebaseError(`Failed to list log entries ${err.message}`, { exit: 1 });
     }
   });