search

nodejs / import-in-the-middle

Like `require-in-the-middle`, but for ESM import
https://www.npmjs.com/package/import-in-the-middle
Apache License 2.0
67 stars 24 forks source link

Regression between import-in-the-middle v1.7.1 and v1.7.2 #61

Closed gajus closed 8 months ago

gajus commented 8 months ago

@opentelemetry debug output in v1.7.2:

Loading instrumentation for @opentelemetry/instrumentation-aws-sdk
Loading instrumentation for @opentelemetry/instrumentation-bunyan
Loading instrumentation for @opentelemetry/instrumentation-cassandra-driver
Loading instrumentation for @opentelemetry/instrumentation-connect
Loading instrumentation for @opentelemetry/instrumentation-cucumber
Loading instrumentation for @opentelemetry/instrumentation-dataloader
Loading instrumentation for @opentelemetry/instrumentation-dns
Loading instrumentation for @opentelemetry/instrumentation-express
Loading instrumentation for @opentelemetry/instrumentation-fastify
Loading instrumentation for @opentelemetry/instrumentation-fs
Loading instrumentation for @opentelemetry/instrumentation-generic-pool
Loading instrumentation for @opentelemetry/instrumentation-graphql
Loading instrumentation for @opentelemetry/instrumentation-grpc
Loading instrumentation for @opentelemetry/instrumentation-hapi
Loading instrumentation for @opentelemetry/instrumentation-http
Loading instrumentation for @opentelemetry/instrumentation-ioredis
Loading instrumentation for @opentelemetry/instrumentation-knex
Loading instrumentation for @opentelemetry/instrumentation-koa
Loading instrumentation for @opentelemetry/instrumentation-lru-memoizer
Loading instrumentation for @opentelemetry/instrumentation-memcached
Loading instrumentation for @opentelemetry/instrumentation-mongodb
Loading instrumentation for @opentelemetry/instrumentation-mongoose
Loading instrumentation for @opentelemetry/instrumentation-mysql2
Loading instrumentation for @opentelemetry/instrumentation-mysql
Loading instrumentation for @opentelemetry/instrumentation-nestjs-core
Loading instrumentation for @opentelemetry/instrumentation-net
Loading instrumentation for @opentelemetry/instrumentation-pg
Loading instrumentation for @opentelemetry/instrumentation-pino
Loading instrumentation for @opentelemetry/instrumentation-redis
Loading instrumentation for @opentelemetry/instrumentation-redis-4
Loading instrumentation for @opentelemetry/instrumentation-restify
Loading instrumentation for @opentelemetry/instrumentation-router
Loading instrumentation for @opentelemetry/instrumentation-socket.io
Loading instrumentation for @opentelemetry/instrumentation-tedious
Loading instrumentation for @opentelemetry/instrumentation-winston
(node:57158) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
@opentelemetry/instrumentation-http Applying patch for http@21.6.0
@opentelemetry/instrumentation-http Applying patch for http@21.6.0
@opentelemetry/instrumentation-http Applying patch for https@21.6.0
@opentelemetry/instrumentation-http Applying patch for https@21.6.0
Applying patch for dns
patch lookup function
patch lookup function
Applying patch for dns
patch lookup function
patch lookup function
@opentelemetry/instrumentation-fs Applying patch for fs
@opentelemetry/instrumentation-fs Applying patch for fs
Applying patch for pino@8.17.2
Applying patch for pino@8.17.2

VS

v1.7.1:

Loading instrumentation for @opentelemetry/instrumentation-aws-sdk
Loading instrumentation for @opentelemetry/instrumentation-bunyan
Loading instrumentation for @opentelemetry/instrumentation-cassandra-driver
Loading instrumentation for @opentelemetry/instrumentation-connect
Loading instrumentation for @opentelemetry/instrumentation-cucumber
Loading instrumentation for @opentelemetry/instrumentation-dataloader
Loading instrumentation for @opentelemetry/instrumentation-dns
Loading instrumentation for @opentelemetry/instrumentation-express
Loading instrumentation for @opentelemetry/instrumentation-fastify
Loading instrumentation for @opentelemetry/instrumentation-fs
Loading instrumentation for @opentelemetry/instrumentation-generic-pool
Loading instrumentation for @opentelemetry/instrumentation-graphql
Loading instrumentation for @opentelemetry/instrumentation-grpc
Loading instrumentation for @opentelemetry/instrumentation-hapi
Loading instrumentation for @opentelemetry/instrumentation-http
Loading instrumentation for @opentelemetry/instrumentation-ioredis
Loading instrumentation for @opentelemetry/instrumentation-knex
Loading instrumentation for @opentelemetry/instrumentation-koa
Loading instrumentation for @opentelemetry/instrumentation-lru-memoizer
Loading instrumentation for @opentelemetry/instrumentation-memcached
Loading instrumentation for @opentelemetry/instrumentation-mongodb
Loading instrumentation for @opentelemetry/instrumentation-mongoose
Loading instrumentation for @opentelemetry/instrumentation-mysql2
Loading instrumentation for @opentelemetry/instrumentation-mysql
Loading instrumentation for @opentelemetry/instrumentation-nestjs-core
Loading instrumentation for @opentelemetry/instrumentation-net
Loading instrumentation for @opentelemetry/instrumentation-pg
Loading instrumentation for @opentelemetry/instrumentation-pino
Loading instrumentation for @opentelemetry/instrumentation-redis
Loading instrumentation for @opentelemetry/instrumentation-redis-4
Loading instrumentation for @opentelemetry/instrumentation-restify
Loading instrumentation for @opentelemetry/instrumentation-router
Loading instrumentation for @opentelemetry/instrumentation-socket.io
Loading instrumentation for @opentelemetry/instrumentation-tedious
Loading instrumentation for @opentelemetry/instrumentation-winston
(node:57196) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
@opentelemetry/instrumentation-http Applying patch for http@21.6.0
@opentelemetry/instrumentation-http Applying patch for http@21.6.0
@opentelemetry/instrumentation-http Applying patch for https@21.6.0
@opentelemetry/instrumentation-http Applying patch for https@21.6.0
Applying patch for dns
patch lookup function
patch lookup function
Applying patch for dns
patch lookup function
patch lookup function
@opentelemetry/instrumentation-fs Applying patch for fs
@opentelemetry/instrumentation-fs Applying patch for fs
Applying patch for pino@8.17.2
Applying patch for pino@8.17.2
@opentelemetry/instrumentation-fastify Applying patch for fastify@4.25.2
@opentelemetry/instrumentation-fastify Patching fastify constructor function
@opentelemetry/instrumentation-fastify Applying patch for fastify@4.25.2
@opentelemetry/instrumentation-fastify Patching fastify constructor function

Looks like v1.7.2 makes so that fastify dependency can no longer be picked up.

gajus commented 8 months ago

Something else is happening here. Cannot repro after re-installing v1.7.3