Gajus Kuizinas comments

Results 766 comments of


                                            Gajus Kuizinas

Using Sentry breadcrumbs and tags alongside OpenTelemetry isn't working correctly

@seeARMS How are you getting `samplingContext?.request?.url`? I am only getting: ```ts { samplingContext: { transactionContext: { name: 'GET', parentSampled: undefined }, parentSampled: undefined } } ``` https://github.com/getsentry/sentry-javascript/discussions/10399#discussion-6149696 and looking at...

MaxListenersExceededWarning: Possible EventEmitter memory leak detected

I was able to replicate the issue. Investigating.

MaxListenersExceededWarning: Possible EventEmitter memory leak detected

Opened an issue with Jest. I am not sure what is causing the memory leak warning or how to mitigate it. https://github.com/jestjs/jest/issues/14580

test: replicate memory leak (#79)

Removing binding on the event emitter eliminates the error. ```diff import { type LogWriter } from '../types'; const createBlockingWriter = (stream: NodeJS.WritableStream): LogWriter => { return (message: string) => {...

test: replicate memory leak (#79)

Asked related question https://stackoverflow.com/questions/77190473/why-is-assigning-a-listener-to-process-stdout-error-event-produce-a-memory-lea

Document POSTGRES_SEEDS (or better yet, use a descriptive var name)

Unbelievable. @colelawrence This prob saved me hours of search 🤦

Cannot use import statement outside a module

This could be solved by providing an ESM build.

Cannot use import statement outside a module

Ah, I looked at the wrong dist file.

CommonJS register/unregister methods

Related https://github.com/privatenumber/tsx/discussions/461 ?

Rename @segment/analytics-node to @segment/analytics-javascript

> I've even mulled the idea of retaining analytics-node and adding another library like 'analytics-javascript' (like a base library) or 'analytics-web-worker' (maybe just a thin wrapper around analytics node). This...