cloudevents / cloudevents/sdk-java
Avro compact deserializer bug - ID set to type
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Java
- Sterne
- 446
- Forks
- 172
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Looks like accidental set of ID to type.
https://github.com/cloudevents/sdk-java/blob/main/formats/avro-compact/src/main/java/io/cloudevents/avro/compact/AvroCompactFormat.java#L90
Should be:
.withId(from.getId())
Test should be fixed as well. Using different values in type and ID of the test event would detect the issue.
https://github.com/cloudevents/sdk-java/blob/main/formats/avro-compact/src/test/java/io/cloudevents/avro/compact/AvroCompactFormatTest.java#L46
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginnen Sie mit formats/avro-compact/src/main/java/io/cloudevents/avro/compact/AvroCompactFormat.java ungefähr bei Zeile 90 und sehen Sie sich anschließend den zugehörigen Test in formats/avro-compact/src/test/java/io/cloudevents/avro/compact/AvroCompactFormatTest.java ungefähr bei Zeile 46 an. Aktualisieren Sie die ID-Behandlung wie angegeben und verwenden Sie im Test unterschiedliche Typ- und ID-Werte, damit der Fehler erkannt wird; führen Sie die Tests für das Avro compact format aus, um die Behebung zu bestätigen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java
- Bereich
- backend
- Issue-Typ
- Bug
- Schwierigkeit
- 1/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Ruhig
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 88/100