Kafka `to_binary()` crashes on events without `datacontenttype` or with non-string attributes
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 78/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Tranquilo
- Stack tecnológico
- kafka, python
- Área
- distributed-systems
Línea de trabajo
Comienza en cloudevents/v1/kafka/conversion.py y compara to_structured() con el binding de Kafka core más reciente mencionado en el issue. Verifica que ambas reproducciones se completen sin excepciones, que se omita content-type cuando datacontenttype esté ausente y que los atributos de extensión que no sean cadenas se conviertan a cadenas en los headers generados.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Python
- Estrellas
- 342
- Forks
- 65
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de cloudevents/sdk-python
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
cloudevents/sdk-python#300 ·
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
cloudevents/sdk-python#291 · 8 comentarios ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 48/100
cloudevents/sdk-python#247 · 2 comentarios ·
-
question
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
cloudevents/sdk-python#246 · 5 comentarios ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
cloudevents/sdk-python#213 · 3 comentarios · 1 reacción ·
Todos los issues de cloudevents/sdk-python
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
stephrobert/dsoxlab#238 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
sublimehq/package_control#1780 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
nwg-piotr/nwg-displays#145 ·