improve logs & capi event source URL handling
This commit is contained in:
@@ -12,7 +12,12 @@ import { resolveGetter, type MaybeGetter } from '../util/getter.ts';
|
||||
import log from 'loglevel';
|
||||
import { dev } from '$app/environment';
|
||||
import { CAPIClient } from '../capi/client.ts';
|
||||
import { ActionSource, CAPIEvent, type CAPIStandardParams } from '../capi/event.ts';
|
||||
import {
|
||||
ActionSource,
|
||||
CAPIEvent,
|
||||
type CAPIEventOptions,
|
||||
type CAPIStandardParams
|
||||
} from '../capi/event.ts';
|
||||
|
||||
const pixelParamsToCustomData = (params: CommonParams & CustomParams): CAPIStandardParams => {
|
||||
const customData: CAPIStandardParams = {};
|
||||
@@ -101,7 +106,7 @@ export class PixelControl {
|
||||
this._conversionClient = new CAPIClient(options.conversionHref, resolvedTrackingManager);
|
||||
} else if (options?.conversionHref) {
|
||||
log.warn(
|
||||
`[PixelControl] Conversion Client ${options.conversionHref} for Meta Pixel [${this._pixelID}] not initialized, TrackingManager is required for user consent.`
|
||||
`${this.logPrefix} CAPI Client ${options.conversionHref} available but not initialized, TrackingManager is required for user consent.`
|
||||
);
|
||||
}
|
||||
}
|
||||
@@ -150,7 +155,7 @@ export class PixelControl {
|
||||
// Check for existing PixelControl instance
|
||||
if (this._registeredPixels[pixelID]) {
|
||||
log.warn(
|
||||
`[PixelControl] Instance for Meta Pixel ID: ${pixelID} already exists. Returning existing instance.`
|
||||
`${this._registeredPixels[pixelID].logPrefix} Instance already exists. Returning existing instance.`
|
||||
);
|
||||
return this._registeredPixels[pixelID];
|
||||
}
|
||||
@@ -161,7 +166,7 @@ export class PixelControl {
|
||||
|
||||
// Fire initialization
|
||||
window.fbq('init', pixel._pixelID, options?.advancedMatching, options?.initOptions);
|
||||
log.debug(`[PixelControl] [${pixel._pixelID}] initialized.`);
|
||||
log.debug(`${pixel.logPrefix} initialized.`);
|
||||
|
||||
return pixel;
|
||||
}
|
||||
@@ -196,12 +201,17 @@ export class PixelControl {
|
||||
private devModeWarn() {
|
||||
if (dev && !this._testEventCode) {
|
||||
log.warn(
|
||||
`[PixelControl] [${this._pixelID}] Sending events in dev mode without a test event code. ` +
|
||||
`${this.logPrefix} Sending events in dev mode without a test event code. ` +
|
||||
'Consider providing a test event code to avoid affecting real data.'
|
||||
);
|
||||
}
|
||||
}
|
||||
|
||||
/** Returns the log prefix including the pixel ID */
|
||||
private get logPrefix(): string {
|
||||
return `[PixelControl] [${this._pixelID}]`;
|
||||
}
|
||||
|
||||
/**
|
||||
* Shorthand utility to send a PageView event
|
||||
* @param disableCAPI If true, disables sending this event to the Conversion API
|
||||
@@ -230,23 +240,28 @@ export class PixelControl {
|
||||
eventID = crypto.randomUUID();
|
||||
}
|
||||
|
||||
const opts: CAPIEventOptions = {
|
||||
eventName: event,
|
||||
eventID: eventID,
|
||||
actionSource: ActionSource.Website,
|
||||
eventSourceURL: window.location.href,
|
||||
eventTime: new Date(),
|
||||
userData: {
|
||||
clientUserAgent: navigator.userAgent
|
||||
},
|
||||
customData: params ? pixelParamsToCustomData(params) : undefined
|
||||
};
|
||||
const ev = CAPIEvent.fromOpts(opts);
|
||||
|
||||
log.debug(
|
||||
`${this.logPrefix} ${ev.readableName} forwarding to CAPI with body: ${JSON.stringify(ev.params, null, 2)}`
|
||||
);
|
||||
|
||||
this._conversionClient
|
||||
.trackEvent(
|
||||
CAPIEvent.fromOpts({
|
||||
eventName: event,
|
||||
eventID: eventID,
|
||||
actionSource: ActionSource.Website,
|
||||
eventSourceURL: window.location.href,
|
||||
eventTime: new Date(),
|
||||
userData: {
|
||||
clientUserAgent: navigator.userAgent
|
||||
},
|
||||
customData: params ? pixelParamsToCustomData(params) : undefined
|
||||
})
|
||||
)
|
||||
.trackEvent(ev)
|
||||
.then((response) => {
|
||||
log.debug(
|
||||
`[PixelControl] [${this._pixelID}] ${event} event forwarded to Conversion API with Event ID: ${eventID}, Response: ${JSON.stringify(
|
||||
`${this.logPrefix} ${ev.readableName} forwarded to CAPI, response: ${JSON.stringify(
|
||||
response,
|
||||
null,
|
||||
2
|
||||
@@ -254,15 +269,22 @@ export class PixelControl {
|
||||
);
|
||||
})
|
||||
.catch((error) => {
|
||||
log.error(
|
||||
`[PixelControl] [${this._pixelID}] Failed to forward ${event} event to Conversion API with Event ID: ${eventID}`,
|
||||
error
|
||||
);
|
||||
log.error(`${this.logPrefix} ${ev.readableName} failed to forward to CAPI. Error: `, error);
|
||||
});
|
||||
|
||||
return eventID;
|
||||
}
|
||||
|
||||
private getEventReadableName(
|
||||
event: StandardEventName | string,
|
||||
eventID?: string,
|
||||
testCode?: string
|
||||
): string {
|
||||
return (
|
||||
event + (eventID ? ` (${eventID})` : '') + (testCode ? ` [test_event_code: ${testCode}]` : '')
|
||||
);
|
||||
}
|
||||
|
||||
/**
|
||||
* Tracks a standard event for this pixel (uses `trackSingle` under the hood)
|
||||
* @param event Standard event name
|
||||
@@ -291,7 +313,7 @@ export class PixelControl {
|
||||
test_event_code: this._testEventCode
|
||||
});
|
||||
log.debug(
|
||||
`[PixelControl] [${this._pixelID}] ${event} event sent with event ID ${eventID} ${dev && ` (test code: ${this._testEventCode})`}.`
|
||||
`${this.logPrefix} ${this.getEventReadableName(event, eventID, this._testEventCode)} sent.`
|
||||
);
|
||||
}
|
||||
|
||||
@@ -323,7 +345,7 @@ export class PixelControl {
|
||||
test_event_code: this._testEventCode
|
||||
});
|
||||
log.debug(
|
||||
`[PixelControl] [${this._pixelID}] ${event} custom event sent with event ID ${eventID} ${dev && ` (test code: ${this._testEventCode})`}.`
|
||||
`${this.logPrefix} ${this.getEventReadableName(event, eventID, this._testEventCode)} sent.`
|
||||
);
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user