Validate with official schema, sanitize identifiers properly, panic on duplicate operationId

[stano45]

No description provided.

[doepnern]

Sieht gut aus finde ich.
Der unique check für operationId ist auch gut, wenn man in das offizielle schema guckt, dann ist operationId gar nicht required. Also können wir das auch gar nicht standardmäßig als Identifier nehmen. Vielleicht wäre es schlau da dann als fallback oder so channelName +"" + subscribe/publish zu nehmen

[bird-dancer]

Wenn du die validator json Dateien, nicht komplett selbst geschrieben hast, dann muss da denke ich noch ein copyright notice hin

[doepnern]

Das ist ja von der offizielen asyncapi-initiative https://github.com/asyncapi/spec-json-schemas/tree/master/schemas, müssen wir nicht eh die zitieren da wir ein programm basierend auf deren spec schreiben

[bird-dancer]

Das ist ja von der offizielen asyncapi-initiative https://github.com/asyncapi/spec-json-schemas/tree/master/schemas, müssen wir nicht eh die zitieren da wir ein programm basierend auf deren spec schreiben

Ja, aber deren spec project inklusive, der json files ist unter der apache2.0 Lizenz veröffentlicht, und die fordert attribution für unveränderte code Bestandteile. Also nicht nur im Bezug auf Zitate in der Abgabe, sondern es wäre denke ich eine Urheberrechtsverletzung.

[bird-dancer]

sanitize_operation_ids_and_check_duplicate() could be done more elegantly with iterators, but its fine for now.
looks great 😄