Kafka `to_binary()` crashes on events without `datacontenttype` or with non-string attributes
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 78/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Calme
- Stack technique
- kafka, python
- Domaine
- distributed-systems
Piste de recherche
Commencez dans cloudevents/v1/kafka/conversion.py et comparez to_structured() avec le binding Kafka core plus récent mentionné dans l’issue. Vérifiez que les deux reproductions se terminent sans exception, que content-type soit omis lorsque datacontenttype est absent et que les attributs d’extension qui ne sont pas des chaînes soient convertis en chaînes dans les en-têtes générés.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Description
to_binary() in cloudevents/v1/kafka/conversion.py raises on two kinds of perfectly valid CloudEvents:
1. Missing datacontenttype (optional attribute) → KeyError
headers = {}
if event["datacontenttype"]: # KeyError if unset
headers["content-type"] = event["datacontenttype"].encode("utf-8")
datacontenttype is optional, so event["datacontenttype"] raises KeyError for any event that does not set it. The sibling to_structured() in the same file already guards this with if "datacontenttype" in attrs:.
2. Non-string attribute value → AttributeError
if value is not None:
headers["ce_{0}".format(attr)] = value.encode("utf-8") # AttributeError if not str
CloudEvents extension attributes may be non-string (e.g. Integer/Boolean), so value.encode(...) raises AttributeError: 'int' object has no attribute 'encode'. The newer core Kafka binding already does str(attr_value).encode(...).
Reproduction
from cloudevents.v1.http import CloudEvent
from cloudevents.v1.kafka import to_binary
# 1) no datacontenttype -> KeyError: 'datacontenttype'
to_binary(CloudEvent({"type": "t", "source": "s"}, {"a": 1}))
# 2) non-string extension -> AttributeError: 'int' object has no attribute 'encode'
to_binary(CloudEvent({"type": "t", "source": "s",
"datacontenttype": "application/json", "ext1": 5}, {"a": 1}))
Expected behaviour
to_binary() should handle both valid events: omit the content-type header when datacontenttype is absent, and stringify non-string attribute values.
- Langage dominant
- Python
- Étoiles
- 343
- Forks
- 65
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un 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 cloudevents/sdk-python
-
CloudEvent serializes invalid extension attribute names without validationPeut-être pris @MickeyWzt l’a pris il y a 102 jours. Ouvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
cloudevents/sdk-python#300 ·
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
cloudevents/sdk-python#291 · 8 commentaires ·
-
Improve the docstring of abstract.CloudEventPeut-être pris @manduinca l’a pris il y a 86 jours. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 48/100
cloudevents/sdk-python#247 · 2 commentaires ·
-
question
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
cloudevents/sdk-python#246 · 5 commentaires ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
cloudevents/sdk-python#213 · 3 commentaires · 1 réaction ·
Toutes les issues de cloudevents/sdk-python
Issues similaires
-
[Bug]: LangGraph Store sync put bypasses namespace validationPeut-être pris @ktz03 l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
volcengine/OpenViking#6050 ·
Les mainteneurs répondent en général sous 1 jour
-
[Bug]: Streamed /v1/chat/completions sends unredacted agent error text (keys) in the final chunkPeut-être pris @rodricksz4h5 l’a pris aujourd’hui. Ouvertearea/streaming comp/gateway P2 type/security
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
NousResearch/hermes-agent#138245 ·
Les mainteneurs répondent en général sous 1 jour
-
[oblt-aw][autodoc] Update user onboarding guide for mandatory trigger CODEOWNERS stepPeut-être pris @github-actions l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
Les mainteneurs répondent en général sous 1 jour
-
correction metadata
Difficulté 2/5 Moins d'une heure Accessibilité débutants 68/100
acl-org/acl-anthology#10378 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
cloudtrail_multi_region_enabled_logging_management_events fails when the advanced event selector has a custom namePeut-être pris @12YYCCYY16 l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
prowler-cloud/prowler#13033 ·
Les mainteneurs répondent en général sous 1 jour