Don't log downsample warning when unnecessary (#4291)

* fix log

use process.env

ensure the log happens regardless of `isDownsampledSession`

fix log

* oops, revert test
zio/stable
Hailey 2024-05-30 17:52:56 -07:00 committed by GitHub
parent d6275e98c2
commit 8e2c21a882
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 10 additions and 7 deletions

View File

@ -102,16 +102,19 @@ export function logEvent<E extends keyof LogEvents>(
rawMetadata: LogEvents[E] & FlatJSONRecord,
) {
try {
if (
process.env.NODE_ENV === 'development' &&
eventName.endsWith(':sampled') &&
!DOWNSAMPLED_EVENTS.has(eventName)
) {
logger.error(
'Did you forget to add ' + eventName + ' to DOWNSAMPLED_EVENTS?',
)
}
if (isDownsampledSession && DOWNSAMPLED_EVENTS.has(eventName)) {
return
}
if (process.env.NODE_ENV === 'development') {
if (eventName.endsWith(':sampled')) {
logger.error(
'Did you forget to add ' + eventName + ' to DOWNSAMPLED_EVENTS?',
)
}
}
const fullMetadata = {
...rawMetadata,
} as Record<string, string> // Statsig typings are unnecessarily strict here.