gematik / api-kim

API specification for gematik's KIM - a secure Email standard, which enables the exchange of information and payload in the German healthcare sector.
Other
33 stars 12 forks source link

Integration der 'KIM-Encounter-Id' in KIM-Spezifikation #69

Closed SvenSommer closed 10 months ago

SvenSommer commented 10 months ago

Änderungsdetails:

Ziel der Änderung:

Verbesserung der Nachverfolgbarkeit und konsistenten Zuordnung von Nachrichten zu spezifischen Behandlungsfällen.

Amarteau commented 10 months ago

KOM-LE-A_2098-03 - Header der äußeren Nachricht schreibt vor:

Ein Headerelement der inneren Nachricht beginnend mit dem Bezeichner X-KIM- MUSS in den äußeren Header
übernommen werden.

Somit wäre der Header X-KIM-Encounter-Id nicht nur Teil der verschlüsselten, inneren KIM-Nachricht sondern auch Teil der äußeren Nachricht. Ist es wirklich notwendig Informationen mit konkretem Patientenbezug unverschlüsselt in der äußeren Nachricht zu transportieren?

dhufnagel commented 10 months ago

@Amarteau hat einen validen Punkt. Ggf. wäre ein Header Name, der mit einem anderen Präfix beginnt für solche Zwecke sinnvoller. Andererseits gibt es für „Konversation“ ja schon die messageId und den In-Reply-To Header. Den Verlauf müsste dann aber natürlich das PVS zu einem Behandlungsfall mappen.

dhufnagel commented 10 months ago

Scahde dass es kein Feedback zum Feedback gibt.. warum kommentiert man hier eigentlich?! 🤷‍♂️

Amarteau commented 10 months ago

In f64c580da1d1a3157222ee6fd4b435da29ed4681 im develop wurde der Header nun umbenannt. Somit wird er nicht mehr in die äußere Nachricht übernommen.

SvenSommer commented 10 months ago

Vielen Dank an @dhufnagel und @Amarteau für die konstruktiven Beiträge,

SvenSommer commented 10 months ago

Die Zusammenfassung der in der heutigen E-Rezept Sprechstunde diskutierten Bedenken und Unterstützungen bezüglich des vorgeschlagenen 'KIM-Encounter-Id' Headers sieht folgendermaßen aus:

Bedenken:

Unterstützung:

Nach meiner aktuellen Einschätzung überwiegen die Bedenken bezüglich der Implementierung eines neuen Attributs für die Darstellung von Nachrichtenverläufen, da die erforderlichen Funktionen bereits in den vorhandenen Standards abgedeckt sind.

Welchen klaren Vorteil bietet die 'KIM-Encounter-Id' gegenüber einem Identifikator im 'Reply-To'?

dhufnagel commented 10 months ago

Genau das ist meine Kritik hierbei: vorschnelles mergen ohne die Diskussionen zu beachten und darauf einzugehen. Die Bedenken sind absolut nachvollziehbar und ein extra Header für bereits vorhandene Header ist unnötig. Schaut man mal rechts und links, gibt es schon seit Jahrzehnten Mailing-Listen, die eben genau jenes Feature nutzen und in den letzten Jahren hat es die Conversation Ansicht auch in Enduser Anwendungen wie Outlook und Thunderbird geschafft. Dabei wird übrigens nicht nur In-Reply-To verwendet, sondern auch „References“, was 1:1 dem Zweck des neuen Headers entsprechen würde. https://datatracker.ietf.org/doc/html/rfc5322#section-3.6.4

SvenSommer commented 10 months ago

Nach eingehender Betrachtung der Diskussionen und Bedenken, die im Kontext des vorgeschlagenen 'KIM-Encounter-Id' Headers aufgekommen sind, möchte ich auf einen neuen Pull Request verweisen, der eine alternative Lösung präsentiert. Dieser neue PR Pull Request #72 adressiert die hervorgebrachten Bedenken bezüglich der Kompatibilität mit dem MIME-Standard und den potenziellen Auswirkungen auf etablierte Tools wie Thunderbird.

In Übereinstimmung mit den Anmerkungen von @dhufnagel und anderen Beitragenden haben wir uns entschieden, statt eines neuen, spezifischen Headers die etablierten MIME-Header 'Reply-To', 'In-Reply-To' und 'References' zu nutzen. Diese Anpassung entspricht den gängigen Kommunikationsstandards, wie in RFC 822 beschrieben, und gewährleistet eine effiziente Nachrichtenverfolgung in bestehenden Systemen, ohne zusätzliche Komplexität oder Kompatibilitätsprobleme zu verursachen.

Der neue PR schlägt eine strukturierte Nutzung dieser Header für die Organisation von Nachrichten in Konversationsverläufen vor. Diese Methode ist nicht nur MIME-konform, sondern auch besser geeignet, um die Bedürfnisse der Nutzer zu erfüllen, die auf eine reibungslose Integration in Standard-E-Mail-Clients angewiesen sind. Wir glauben, dass dieser Ansatz eine ausgewogene und praktikable Lösung darstellt, die sowohl die technischen als auch die benutzerorientierten Aspekte berücksichtigt.

Ich lade alle Beteiligten ein, den neuen Pull Request Pull Request #72 zu überprüfen und Feedback zu geben. Dieser Vorschlag soll die Diskussionen und Bedenken, die bisher geäußert wurden, aufgreifen und eine zufriedenstellende Lösung für alle Beteiligten bieten.