Skip to content

Commit

Permalink
Re-disable logevent compression (#403)
Browse files Browse the repository at this point in the history
This was explicitly disabled in v5.12.1 and v5.13.2. Then it was moved
to an SDK flag without fixing the underlying problem. We should not rely
on an sdk flag for functionality that is definitively incorrect. Lets
remove this flag check entirely until the underlying issue is fixed. We
will likely re-release using a different flag
  • Loading branch information
tore-statsig authored Jan 12, 2024
1 parent 31ffd26 commit 904d77c
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions src/LogEventProcessor.ts
Original file line number Diff line number Diff line change
Expand Up @@ -102,14 +102,12 @@ export default class LogEventProcessor {
events: oldQueue,
};

const isCompressionDisabled = SDKFlags.on('stop_log_event_compression');

return this.fetcher
.post(this.explicitOptions.api + '/log_event', body, {
retries: fireAndForget ? 0 : this.explicitOptions.postLogsRetryLimit,
backoff: this.explicitOptions.postLogsRetryBackoff,
signal: abortSignal,
compress: !isCompressionDisabled,
compress: false,
})
.then(() => {
return Promise.resolve();
Expand Down

0 comments on commit 904d77c

Please sign in to comment.