Better Error handling for empty token list in Messaging send_each_multicast

Offen

#792 erstellt am 17.6.2024 1 Kommentar 4 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 2/5
Geschätzter Aufwand: 1-3 Stunden
Anfängerfreundlichkeit: 35/100

Issue-Typ: Bug
Klarheit: Klar beschrieben
Aktivitätsstatus: Veraltet
Tech-Stack: python
Bereich: api

Rechercherichtung

Beginne in firebase_admin/messaging.py bei send_each_multicast, ungefähr bei der verlinkten Zeile, und verfolge, wie eine leere Gerätetoken-Liste send_each und den Thread-Pool erreicht. Vergleiche die alternative Stelle in send_each. Erledigt ist es, wenn eine leere Liste einen beschreibenden ValueError statt des aktuellen Fehlers wegen unbekanntem max_workers erzeugt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

api: messaging type: feature request

Problem
Non-descriptive error message returned when sending multicast message with empty device token list. Currently the empty list of tokens propagates throughout the code and results in an Unknown Error type with a vague error message as follows: "Unknown error while making remote service calls: max_workers must be greater than 0". This error seems to originate here as the Threadpool is created with max_workers based on the length of the messages parameter, which is ultimately dependent on the number of tokens in the Message object.

Describe the solution you'd like
Check the length of device tokens directly in the send_each_multicast method and return a descriptive ValueError in the event of an empty list. here

Describe alternatives you've considered
Check the length of the message directly in the send_each class method here.

Vorherrschende Sprache: Python
Sterne: 1.2k
Forks: 359
Ø Merge: 3 T. 9 Std.
Gemergte PRs (30 T.): 3

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus firebase/firebase-admin-python

Fix typos and wrong types in remote_config.py Offen

api: remoteconfig

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100

firebase/firebase-admin-python#957 · 1 Kommentar · Erstellt am 27.6.2026
FR: Make the Number of Retires Configurable in transaction() Offen

api: database type: feature request

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100

firebase/firebase-admin-python#58 · Erstellt am 14.8.2017
"Invalid database id (default)" con Firestore Python (Admin SDK) Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100

firebase/firebase-admin-python#978 · 1 Kommentar · Erstellt am 25.8.2026
[FR] Support VERIFY_AND_CHANGE_EMAIL in generate_email_action_link (parity with firebase-admin-node) Offen

api: auth

firebase/firebase-admin-python#949 · 2 Kommentare · 1 Reaktion · 1 zugewiesene Person · Erstellt am 23.4.2026
ECONNRESET connecting to Firebase auth emulator from functions emulator with grpc imported Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 43/100

firebase/firebase-admin-python#945 · 1 Kommentar · 1 Reaktion · Erstellt am 14.4.2026

Alle Issues in firebase/firebase-admin-python