aws / aws/aws-xray-sdk-python

patch_all() is unsafe

Offen
#100 1 Kommentar 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen
discussion enhancement
Vorherrschende Sprache
Python
Sterne
338
Forks
147
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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()`

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne damit, die Python-SDK-Implementierung und die Aufrufer von patch_all() zu lokalisieren. Lies anschließend die verlinkten Issues #48 und #90 und überprüfe den hier beschriebenen Fehler bei der Lambda-Bereitstellung. Als erledigt gilt die Aufgabe erst, wenn eine von einem Maintainer ausgewählte Richtlinie für neue Patcher sowie eine vereinbarte Änderung an der API oder am Verhalten vorliegen; dieses Issue nennt keine Dateien oder Tests.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
observability
Issue-Typ
Feature
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.