[restful-ws] Allow configuring default writer encoding for CloudEventsProvider

Offen

#533 erstellt am 03.3.2023 4 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 4/5
Geschätzter Aufwand: 3-5 Tage
Anfängerfreundlichkeit: 45/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: java
Bereich: api

Rechercherichtung

Beginne in http/restful-ws/src/main/java/io/cloudevents/http/restful/ws/CloudEventsProvider.java, insbesondere mit der Logik rund um die Zeilen 95–112, und überprüfe das verknüpfte Quarkus-SSE-Verhalten. Ermittle, wie eine konfigurierbare Standardkodierung mit den Annotationen StructuredEncoding und BinaryEncoding interagieren sollte. Die Aufgabe ist erledigt, wenn Aufrufer den Standardmodus auswählen können und das Kodierungsverhalten pro Methode weiterhin sinnvoll bleibt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

enhancement

The CloudEventsProvider defaults to binary encoding. Only when there is an StructuredEncoding annotation present, this mode will be set. See the logic:

https://github.com/cloudevents/sdk-java/blob/3614a4f5f4982e6ed38332c2cc73b1a558326274/http/restful-ws/src/main/java/io/cloudevents/http/restful/ws/CloudEventsProvider.java#L95-L112

This makes the BinaryEncoding useless, confirmed by no usages in the code.

The default to the binary mode is unfortunate. Some implementations like Quarkus RESTEasy Reactive Server Sent Events (SSE) don't provide the annotations. This is problematic, as the binary mode doesn't make sense in SSE - the headers set by CloudEventsProvider are being dropped.

See the code:

https://github.com/quarkusio/quarkus/blob/247736226a8c8a55fa88a662eda05963803905eb/independent-projects/resteasy-reactive/server/runtime/src/main/java/org/jboss/resteasy/reactive/server/core/SseUtil.java#L141-L142

See repro: https://github.com/cardil/openshift-knative-showcase/commit/15e00429c459abf655fb483a27f467d598ae173e
Related issue: https://github.com/quarkusio/quarkus/issues/31559

The fact the Quarkus doesn't pass the method annotations, is probably a bug in Quarkus. But, I feel the CloudEvents SDK should allow setting the default mode. Then, the BinaryEncoding could have a meaning, of changing that default encoding per-method.

/kind bug

Vorherrschende Sprache: Java
Sterne: 446
Forks: 172
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

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 cloudevents/sdk-java

Supported Features are not up to date Offen

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

cloudevents/sdk-java#731 · 1 Reaktion · Erstellt am 15.5.2026
Avro compact deserializer bug - ID set to type Offen

bug

Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 88/100

cloudevents/sdk-java#697 · 3 Kommentare · Erstellt am 23.6.2025
Spring CloudEventMessageConverter produce a null pointer exception if initial message header map is null Offen

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

cloudevents/sdk-java#689 · 2 Kommentare · Erstellt am 02.4.2025
JsonFormat doesn't accept valid datacontentypes Offen

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

cloudevents/sdk-java#660 · 2 Kommentare · Erstellt am 11.7.2024
[Vulnerability] jackson-core to version 2.18.6 Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 48/100

cloudevents/sdk-java#711 · 1 Kommentar · Erstellt am 23.3.2026

Alle Issues in cloudevents/sdk-java