Bluesky app fork with some witchin' additions 💫

Don't send any debugs to Sentry (#8533)

* Don't send any debugs to Sentry

* Fix tests

authored by

Eric Bailey and committed by
GitHub
bc95c0e5 aeafb14f

+9 -4
+5 -3
src/logger/__tests__/logger.test.ts
··· 110 const timestamp = Date.now() 111 const sentryTimestamp = timestamp / 1000 112 113 sentryTransport( 114 LogLevel.Debug, 115 Logger.Context.Default, ··· 125 level: LogLevel.Debug, 126 timestamp: sentryTimestamp, 127 }) 128 129 sentryTransport( 130 LogLevel.Info, ··· 154 message, 155 data: {__context__: 'logger'}, 156 type: 'default', 157 - level: 'debug', // Sentry bug, log becomes debug 158 timestamp: sentryTimestamp, 159 }) 160 jest.runAllTimers() ··· 220 const sentryTimestamp = timestamp / 1000 221 222 sentryTransport( 223 - LogLevel.Debug, 224 undefined, 225 message, 226 {error: new Error('foo')}, ··· 230 message, 231 data: {error: 'Error: foo'}, 232 type: 'default', 233 - level: LogLevel.Debug, 234 timestamp: sentryTimestamp, 235 }) 236 })
··· 110 const timestamp = Date.now() 111 const sentryTimestamp = timestamp / 1000 112 113 + /* 114 sentryTransport( 115 LogLevel.Debug, 116 Logger.Context.Default, ··· 126 level: LogLevel.Debug, 127 timestamp: sentryTimestamp, 128 }) 129 + */ 130 131 sentryTransport( 132 LogLevel.Info, ··· 156 message, 157 data: {__context__: 'logger'}, 158 type: 'default', 159 + level: 'log', 160 timestamp: sentryTimestamp, 161 }) 162 jest.runAllTimers() ··· 222 const sentryTimestamp = timestamp / 1000 223 224 sentryTransport( 225 + LogLevel.Info, 226 undefined, 227 message, 228 {error: new Error('foo')}, ··· 232 message, 233 data: {error: 'Error: foo'}, 234 type: 'default', 235 + level: LogLevel.Info, 236 timestamp: sentryTimestamp, 237 }) 238 })
+4 -1
src/logger/transports/sentry.ts
··· 1 import {isNetworkError} from '#/lib/strings/errors' 2 import {Sentry} from '#/logger/sentry/lib' 3 - import {LogLevel, Transport} from '#/logger/types' 4 import {prepareMetadata} from '#/logger/util' 5 6 export const sentryTransport: Transport = ( ··· 10 {type, tags, ...metadata}, 11 timestamp, 12 ) => { 13 const meta = { 14 __context__: context, 15 ...prepareMetadata(metadata),
··· 1 import {isNetworkError} from '#/lib/strings/errors' 2 import {Sentry} from '#/logger/sentry/lib' 3 + import {LogLevel, type Transport} from '#/logger/types' 4 import {prepareMetadata} from '#/logger/util' 5 6 export const sentryTransport: Transport = ( ··· 10 {type, tags, ...metadata}, 11 timestamp, 12 ) => { 13 + // Skip debug messages entirely for now - esb 14 + if (level === LogLevel.Debug) return 15 + 16 const meta = { 17 __context__: context, 18 ...prepareMetadata(metadata),