Refactor Envelope build & sending
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Lee la SDK envelope specification y, después, inspecciona las APIs de transport, Event, Envelope y Envelope Item mencionadas en el issue. Rastrea cómo se entregan actualmente los eventos a transport e identifica allí el procesamiento específico de los eventos. Se considera terminado cuando transport acepta envelopes y deja de depender de event types; el issue no especifica ninguna prueba concreta.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Summary
Currently, events are handed over to transport for sending. In order to fully switch to Envelopes, transport should not know about events and instead accept envelopes for sending. Transport should also not do any event-type specific processing - all the information transport needs is contained in the Envelope & its Items. See https://develop.sentry.dev/sdk/envelopes/
Motivation
Besides finishing the move to Envelopes, the separation of concerns would allow us to build envelopes with multiple items, without hacking around events, as I've had to do in here - adding a transactionProfile field to Event and handling that in transport.go.
Additional Context
Ideally, I'd also suggest something to think about in the future - changing the APIs to not accept Event struct but rather an interface, sth. like which just knows how to serialize and deserialize itself. This would also make it more flexible and avoid adding fields specific to different event types, e.g. Transactions, as there already are.
- Lenguaje dominante
- Go
- Estrellas
- 1.1k
- Forks
- 264
- Merge medio
- 2 d 6 h
- PR fusionados (30 d)
- 11
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de getsentry/sentry-go
-
ExtractStacktrace resolves the PCs of []runtime.Frame stack traces a second timePosiblemente ocupada @mokevnin la tomó hace 21 días. AbiertoErrors Go
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
getsentry/sentry-go#1461 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
v1 API refinementsAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
getsentry/sentry-go#1485 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Crons: gocron job monitoringAbiertoChecks Crons Feature Go
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
getsentry/sentry-go#1470 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Checks Crons Go Improvement
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
getsentry/sentry-go#1468 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Crons: WithMonitor helperAbiertoChecks Crons Feature Go
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
getsentry/sentry-go#1467 · 1 comentario ·
Los mantenedores suelen responder en 2 días
Todos los issues de getsentry/sentry-go
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
nebari-dev/nebi#652 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
type:bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
nightgauge/nightgauge#3077 ·
Los mantenedores suelen responder en 1 día
-
debug-adapter language server
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
open-policy-agent/regal#2161 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
mattabate/life-distributable#169 ·
Los mantenedores suelen responder en 1 día
-
govulncheck reports 7 reachable vulnerabilities in dependenciesPosiblemente ocupada @dwisiswant0 la tomó hoy. AbiertoType: Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
projectdiscovery/rawhttp#558 ·