Jayden/AFFiNE-Mirror
0
0
Fork 0
mirror of https://github.com/toeverything/AFFiNE.git synced 2026-09-24 04:27:27 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix: SHOULD_REPORT_TRACE condition (#4273)

Browse Source 
Co-authored-by: X1a0t <405028157@qq.com>
This commit is contained in:
Peng Xiao
2023-09-11 09:47:02 +00:00
committed by GitHub
co-authored by X1a0t
parent a25a31c105
commit 892cae5599
4 changed files with 16 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files
apps/core/.webpack/config.ts
+6 -4
View File
@@ -349,10 +349,12 @@ export const createConfiguration: (
'process.env': JSON.stringify({}), 'process.env': JSON.stringify({}),
'process.env.COVERAGE': JSON.stringify(!!buildFlags.coverage), 'process.env.COVERAGE': JSON.stringify(!!buildFlags.coverage),
'process.env.NODE_ENV': JSON.stringify(buildFlags.mode), 'process.env.NODE_ENV': JSON.stringify(buildFlags.mode),
'process.env.SHOULD_REPORT_TRACE': `${Boolean( 'process.env.SHOULD_REPORT_TRACE': JSON.stringify(
process.env.SHOULD_REPORT_TRACE Boolean(process.env.SHOULD_REPORT_TRACE === 'true')
)}`, ),
'process.env.TRACE_REPORT_ENDPOINT': `"${process.env.TRACE_REPORT_ENDPOINT}"`, 'process.env.TRACE_REPORT_ENDPOINT': JSON.stringify(
process.env.TRACE_REPORT_ENDPOINT
),
runtimeConfig: JSON.stringify(runtimeConfig), runtimeConfig: JSON.stringify(runtimeConfig),
}), }),
new CopyPlugin({ new CopyPlugin({
apps/prototype/vite.config.ts
+1 -1
View File
@@ -24,7 +24,7 @@ export default defineConfig({
'process.env': {}, 'process.env': {},
'process.env.COVERAGE': JSON.stringify(!!process.env.COVERAGE), 'process.env.COVERAGE': JSON.stringify(!!process.env.COVERAGE),
'process.env.SHOULD_REPORT_TRACE': `${Boolean( 'process.env.SHOULD_REPORT_TRACE': `${Boolean(
process.env.SHOULD_REPORT_TRACE process.env.SHOULD_REPORT_TRACE === 'true'
)}`, )}`,
'process.env.TRACE_REPORT_ENDPOINT': `"${process.env.TRACE_REPORT_ENDPOINT}"`, 'process.env.TRACE_REPORT_ENDPOINT': `"${process.env.TRACE_REPORT_ENDPOINT}"`,
runtimeConfig: getRuntimeConfig({ runtimeConfig: getRuntimeConfig({
apps/storybook/.storybook/main.ts
+1 -1
View File
@@ -47,7 +47,7 @@ export default {
'process.env': {}, 'process.env': {},
'process.env.COVERAGE': JSON.stringify(!!process.env.COVERAGE), 'process.env.COVERAGE': JSON.stringify(!!process.env.COVERAGE),
'process.env.SHOULD_REPORT_TRACE': `${Boolean( 'process.env.SHOULD_REPORT_TRACE': `${Boolean(
process.env.SHOULD_REPORT_TRACE process.env.SHOULD_REPORT_TRACE === 'true'
)}`, )}`,
'process.env.TRACE_REPORT_ENDPOINT': `"${process.env.TRACE_REPORT_ENDPOINT}"`, 'process.env.TRACE_REPORT_ENDPOINT': `"${process.env.TRACE_REPORT_ENDPOINT}"`,
runtimeConfig: getRuntimeConfig({ runtimeConfig: getRuntimeConfig({
packages/graphql/src/utils.ts
+8 -5
View File
@@ -49,8 +49,7 @@ export function generateRandUTF16Chars(bytes: number) {
} }
export class TraceReporter { export class TraceReporter {
// eslint-disable-next-line @typescript-eslint/no-non-null-assertion static traceReportEndpoint = process.env.TRACE_REPORT_ENDPOINT;
static traceReportEndpoint = process.env.TRACE_REPORT_ENDPOINT!;
static shouldReportTrace = process.env.SHOULD_REPORT_TRACE; static shouldReportTrace = process.env.SHOULD_REPORT_TRACE;
private spansCache = new Array<TraceSpan>(); private spansCache = new Array<TraceSpan>();
@@ -108,6 +107,10 @@ export class TraceReporter {
} }
public static reportToTraceEndpoint(payload: string): void { public static reportToTraceEndpoint(payload: string): void {
if (!TraceReporter.traceReportEndpoint) {
console.warn('No trace report endpoint found!');
return;
}
if (typeof navigator !== 'undefined') { if (typeof navigator !== 'undefined') {
navigator.sendBeacon(TraceReporter.traceReportEndpoint, payload); navigator.sendBeacon(TraceReporter.traceReportEndpoint, payload);
} else { } else {
@@ -201,6 +204,6 @@ export class TraceReporter {
}; };
} }
export const traceReporter = !process.env.SHOULD_REPORT_TRACE export const traceReporter = process.env.SHOULD_REPORT_TRACE
? null ? TraceReporter.getInstance()
: TraceReporter.getInstance(); : null;