Skip to content

Commit d059322

Browse files
committed
chore(server): reduce logging frequency
1 parent 73157b4 commit d059322

File tree

2 files changed

+17
-12
lines changed

2 files changed

+17
-12
lines changed

packages/analytics/src/plugin/with-simple-analytics.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,11 @@ export function withSimpleAnalytics(
3838
): NextConfig {
3939
const hostname = options?.hostname ?? process.env.SIMPLE_ANALYTICS_HOSTNAME;
4040

41+
if (!hostname) {
42+
console.warn("No hostname provided for Simple Analytics, plugin disabled.");
43+
return nextConfig;
44+
}
45+
4146
const clientHints = buildClientHintHeaders(options?.clientHints);
4247

4348
const nextAnalyticsConfig: NextConfig = {

packages/analytics/src/server/simple-analytics.ts

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -23,18 +23,18 @@ export async function trackEvent(
2323
eventName: string,
2424
options: TrackEventOptions,
2525
) {
26-
const hostname = options.hostname ?? process.env.SIMPLE_ANALYTICS_HOSTNAME;
26+
const hostname = options?.hostname ?? process.env.SIMPLE_ANALYTICS_HOSTNAME;
2727

2828
if (!hostname) {
29-
console.error("No hostname provided for Simple Analytics");
29+
// console.error("No hostname provided for Simple Analytics");
3030
return;
3131
}
3232

3333
const headers =
3434
"request" in options ? options.request.headers : options.headers;
3535

3636
if (isDoNotTrackEnabled(headers) && !options.collectDnt) {
37-
console.log("Do not track enabled, not tracking event");
37+
// console.log("Do not track enabled, not tracking event");
3838
return;
3939
}
4040

@@ -51,9 +51,9 @@ export async function trackEvent(
5151
}
5252

5353
if (!isProduction()) {
54-
console.log(
55-
"Simple Analytics is disabled by default in development and preview environments, enable it by setting ENABLE_ANALYTICS_IN_DEV=1 in your environment",
56-
);
54+
// console.log(
55+
// "Simple Analytics is disabled by default in development and preview environments, enable it by setting ENABLE_ANALYTICS_IN_DEV=1 in your environment",
56+
// );
5757
return;
5858
}
5959

@@ -86,10 +86,10 @@ const PROXY_PATHS = /^\/(proxy\.js|auto-events\.js|simple\/.*)$/;
8686
type TrackPageviewOptions = TrackingOptions & ServerContext;
8787

8888
export async function trackPageview(options: TrackPageviewOptions) {
89-
const hostname = options.hostname ?? process.env.SIMPLE_ANALYTICS_HOSTNAME;
89+
const hostname = options?.hostname ?? process.env.SIMPLE_ANALYTICS_HOSTNAME;
9090

9191
if (!hostname) {
92-
console.error("No hostname provided for Simple Analytics");
92+
// console.error("No hostname provided for Simple Analytics");
9393
return;
9494
}
9595

@@ -109,7 +109,7 @@ export async function trackPageview(options: TrackPageviewOptions) {
109109
}
110110

111111
if (isDoNotTrackEnabled(headers) && !options.collectDnt) {
112-
console.log("Do not track enabled, not tracking pageview");
112+
// console.log("Do not track enabled, not tracking pageview");
113113
return;
114114
}
115115

@@ -135,9 +135,9 @@ export async function trackPageview(options: TrackPageviewOptions) {
135135
}
136136

137137
if (!isProduction()) {
138-
console.log(
139-
"Simple Analytics is disabled by default in development and preview environments, enable it by setting ENABLE_ANALYTICS_IN_DEV=1 in your environment",
140-
);
138+
// console.log(
139+
// "Simple Analytics is disabled by default in development and preview environments, enable it by setting ENABLE_ANALYTICS_IN_DEV=1 in your environment",
140+
// );
141141
return;
142142
}
143143

0 commit comments

Comments
 (0)