aws / aws/aws-xray-sdk-python

patch_all() is unsafe

Abierto
#100 1 comentario 1 reacción 0 asignados Ver en GitHub
discussion enhancement
Lenguaje dominante
Python
Estrellas
338
Forks
147
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

There is a bit of ambiguity of `patch_all()` approach on `aws-xray-sdk-python`. Use thereof automatically enables new patchers regardless to how well they've been tested against real use cases (https://github.com/aws/aws-xray-sdk-python/issues/48, https://github.com/aws/aws-xray-sdk-python/issues/90). This in turn can can lead to a nasty surprise where deployment of lambda function will not work as it patches the calls.

Some ideas on making the situation better:
1. Discourage use of `patch_all()`due to its unsafe nature.
2. Do not include new patchers in `patch_all()` outside of major versions
3. Split `patch_all()` into two functions `patch_all()` and `patch_all_bleeding_edge()` where new, and un-vetted patchers are first introduced into `patch_all_bleeding_edge()` and after they've been tested are then introduced to `patch_all()`

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Comienza localizando la implementación del SDK de Python y los llamadores de patch_all(); después, lee las issues enlazadas #48 y #90 y verifica el fallo de despliegue de Lambda descrito aquí. Para darlo por terminado se requiere una política, seleccionada por un maintainer, para los nuevos patchers y un cambio acordado en la API o el comportamiento; esta issue no especifica archivos ni tests.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
python
Área
observability
Tipo de issue
Nueva funcionalidad
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
25/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.