chore: add RC_DISABLE_STATISTICS_REPORTING check to a common function

Author: ricardogarim

apps/meteor/app/statistics/server/functions/sendUsageReport.ts

@@ -6,6 +6,7 @@ import { tracerSpan } from '@rocket.chat/tracing';
 import { Meteor } from 'meteor/meteor';
 
 import { statistics } from '..';
+import { shouldReportStatistics } from '../../../../server/cron/usageReport';
 import { getWorkspaceAccessToken } from '../../../cloud/server';
 
 async function sendStats(logger: Logger, cronStatistics: IStats): Promise<string | undefined> {
@@ -36,7 +37,7 @@ async function sendStats(logger: Logger, cronStatistics: IStats): Promise<string
 export async function sendUsageReport(logger: Logger): Promise<string | undefined> {
 	// Even when disabled, we still generate statistics locally to avoid breaking
 	// internal processes, such as restriction checks for air-gapped workspaces.
-	const shouldSendToCollector = process.env.RC_DISABLE_STATISTICS_REPORTING?.toLowerCase() !== 'true';
+	const shouldSendToCollector = shouldReportStatistics();
 
 	return tracerSpan('generateStatistics', {}, async () => {
 		const last = await Statistics.findLast();

apps/meteor/server/cron/usageReport.ts

@@ -13,11 +13,12 @@ export const sendUsageReportAndComputeRestriction = async (statsToken?: string)
 	void AirGappedRestriction.computeRestriction(token);
 };
 
+export const shouldReportStatistics = () => process.env.RC_DISABLE_STATISTICS_REPORTING?.toLowerCase() !== 'true';
+
 export async function usageReportCron(logger: Logger): Promise<void> {
 	// The actual send suppression happens inside `sendUsageReport`, but since this
 	// is the entry point, we log a warning here when reporting is disabled.
-	const shouldSendToCollector = process.env.RC_DISABLE_STATISTICS_REPORTING?.toLowerCase() !== 'true';
-	if (!shouldSendToCollector) {
+	if (!shouldReportStatistics()) {
 		logger.warn(
 			'Statistics reporting disabled via environment variable (RC_DISABLE_STATISTICS_REPORTING). This may impact product improvements.',
 		);