Weiterleitung an weiteres LRS

Mein Verständnis: Es soll in der Definition der Quell-Systeme ein oder mehrere Endpunkte ergänzt werden, welche eine API und ein Token enthält. Diese API wird für jedes xAPI Statement aufgerufen, welches Polaris empfängt. Eine Filterung oder Validierung innerhalb von Polaris ist nicht vorgesehen, bevor die Weiterleitung der Statements erfolgt.

@thelen bitte erweitern

assigned to @thelen

changed milestone to %Release 1.0.1

Das Ziel der Weiterleitung ist richtig erfasst. Ob die Umsetzung für jede Definition eines Quellsystems geeignet ist muss sich zeigen. Der Forschungsbetrieb kann auf mehrere Quellsysteme ausgeweitet werden und ggf. ist eine zentrale Verwaltung einfacher zu warten und zu erweitern. Das kann ich nicht abschätzen.

mentioned in commit 4efc1b27

mentioned in commit 1cf3130a

assigned to @LenDigLearn and unassigned @thelen

Feedback von Sven muss noch eingearbeitet werden.

Hi Benjamin,

die Modellierung im Commit sieht okay aus. Den Authorization Präfix (Basic, Bearer, Api-Key, ...) brauchen wir aber auch im Schema Konfig. Gerne als drittes Attribut mit einem fixen Wertebereich, um Fehl-Konfigurationen vorzubeugen. Unelegant finde ich das Forwarding. Jedes Statement einzeln zu Verschicken erzeugt einen riesigen Overhead bei Sender und Empfänger und sollte vermieden werden. Zudem braucht es einen besseren Umgang mit nicht akzeptierten Statements. Ist das Ziel ausgelastet oder down, sollte das Verschicken später erneut versucht werden. Gibt es Probleme mit dem Statement, sollte dies der ursprünglich versendenden Instanz zurückgemeldet werden, damit z. B. der Logger das erfahren und darauf reagieren kann.

Beste Grüße Sven

Getan

taper off für erneute versuche

Karenzzeit 2 Wochen, dann entsorgen

perspektivisch ins Monitoring aufnehmen

mentioned in commit f72b5e80

closed

reopened

assigned to @sven.judel and unassigned @LenDigLearn

@sven.judel hattest du schon Zeit dir das anzusehen?

Leider nein. Ich hatte unsere/meine aktuelle Situation Kyra schon erklärt. Wenn es den Rest der Woche so "ruhig" bleibt wie heute und ich mit meinen Projekt ToDos gut voran komme, hab ich Freitag ein Zeitfenster dafür.

Es ist nicht ruhig geblieben. Malte weiß Bescheid.

marked this issue as related to #34

added test label

Der Code sieht okay aus, ich muss aber noch mehr praktisch testen. Dafür konnte ich dank des von Martin angelegten Accounts unter polaris-test.medien.rwth-aachen.de ein Schema hochladen, bekomme im Frontend zwar auch eine Erfolgsmeldung über den Upload, ob die additionalLrs Liste aber übernommen wurde ist mir unklar, die wird unter "Toggle Definition" nicht angezeigt.

Zudem muss das Enum der token types um "Api-Key" erweitert werden.

Ist beides implementiert, hatte versäumt, zu antworten.

assigned to @benjaminLedel

assigned to @LenDigLearn