HttpsError.__init__ does not pass message to super().__init__()
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 1/5
- Temps estimé
- Moins d'une heure
- Accessibilité débutants
- 85/100
Piste de recherche
Le problème se trouve dans src/firebase_functions/https_fn.py au niveau de la méthode HttpsError.__init__ (lignes 251-266). Modifiez l'appel super().__init__() en super().__init__(message) pour transmettre le message d'erreur à la classe Exception parente. Exécutez la suite de tests du module https_fn pour confirmer que str(err) renvoie désormais le message fourni et que err.args est correctement renseigné.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Description
In src/firebase_functions/https_fn.py, HttpsError.__init__ stores self.message = message but calls super().__init__() with no arguments:
def __init__(
self,
code: FunctionsErrorCode,
message: str,
details: _typing.Any | None = None,
):
self.code = code
self.message = message
self.details = details
if code not in _error_code_map:
raise ValueError(f"Unknown error code: ${code}.")
self._http_error_code = _error_code_map[code]
super().__init__()
Impact
Because message is not passed to super().__init__(message):
str(err)evaluates to""(empty string) anderr.argsis().- Logging the exception directly (e.g.
logger.error(err)) or inspecting tracebacks in tests / unhandled contexts printsfirebase_functions.https_fn.HttpsErrorwith no message.
Suggested Fix
Pass message to super().__init__(message).
(Context: surfaced via https://github.com/firebase/functions-samples/issues/1137)
- Langage dominant
- Python
- Étoiles
- 167
- Forks
- 36
- Merge moyen
- 4 j 3 h
- PR mergées (30 j)
- 1
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de firebase/firebase-functions-python
-
Add support for VPC direct connectPeut-être pris @IzaakGough l’a pris il y a 105 jours. Ouvertefeature-request
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
firebase/firebase-functions-python#275 · 1 commentaire · 1 réaction ·
-
feature-request
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
firebase/firebase-functions-python#271 · 3 commentaires ·
-
[Bug] stability_digest_payload_from_ce_payload fails with AttributeError: 'dict' object has no attribute 'split'Peut-être à nouveau libre Une pull request pour cette issue a été fermée sans être fusionnée. Ouvertebug unable to reproduce
Difficulté 3/5 1-2 jours Accessibilité débutants 55/100
firebase/firebase-functions-python#260 · 6 commentaires ·
-
feature-request
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
firebase/firebase-functions-python#249 · 2 commentaires ·
-
logging library doesn't support ExceptionsPeut-être pris @IzaakGough l’a pris il y a 128 jours. Ouvertefeature-request
Difficulté 4/5 3-5 jours Accessibilité débutants 52/100
firebase/firebase-functions-python#172 · 3 commentaires ·
Toutes les issues de firebase/firebase-functions-python
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
-
Clean up dependabot noiseOuverteTask
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 86/100
war-and-code/dircue#200 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 87/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
Les mainteneurs répondent en général sous 1 jour