Re-evaluate RITM externalisation

Aperta
#2,043 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 2 giorni

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
5/5
Tempo stimato
Più di una settimana
Idoneità per principianti
35/100
Tipo di issue
Funzionalità
Chiarezza
Da chiarire
Stato di attività
Tranquilla
Stack tecnologico
node.js, webpack

Direzione di ricerca

Inizia leggendo la PR 2037 e l’issue 120 collegata di require-in-the-middle per comprendere il workaround di esternalizzazione e la soluzione upstream proposta. Il lavoro è completato quando RITM può essere nuovamente incluso nel bundle Node.js Lambda senza rompere la strumentazione che applica hook ai moduli non-core.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

enhancement javascript

Is your feature request related to a problem? Please describe.
This issue is mainly for tracking. In https://github.com/open-telemetry/opentelemetry-lambda/pull/2037 we externalised the require-in-the-middle lib from our webpack bundle for the node.js layer, to fix an issue that broke instrumentation that relied on ritm hooking for non-core modules.

Describe the solution you'd like
Fixing upstream would be ideal, so that we can include RITM in our bundle again. In https://github.com/nodejs/require-in-the-middle/issues/120 I offered some insight as well and proposed a solution that could possibly be implemented in the RITM lib.

Describe alternatives you've considered
Keeping RITM externalised.

Tip: React with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding +1 or me too, to help us triage it. Learn more here.

Lingua principale
Go
Stelle
433
Fork
249
Merge medio
1g 12h
PR unite (30g)
39

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di open-telemetry/opentelemetry-lambda

Tutte le issue di open-telemetry/opentelemetry-lambda

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.