Closed niklasstich closed 1 year ago
Bei gesetztem BackendToken aber leerer BackendBaseUrl in ApplicationConfig stürzt das Programm bei Öffnen des LmsLoginDialog ab.
Leere BackendBaseUrl und beliebigen String in BackendToken in ApplicationConfig.json abspeichern, Autorentool starten, LmsLoginDialog öffnen.
Es wird nicht versucht, den Token über das Backend zu verifizieren, sondern der Token wird einfach gecleared.
Wir versuchen einen Request an eine leere URI zu senden um den Token zu verifizieren.
@andreasweishaupt @niklasstich @philgei
Kein Check in BusinessLogic.UpdateUserInformation ob BackendBaseUrl gesetzt ist.
Check ob BackendBaseUrl gesetzt ist, wenn nicht Logout() und frühzeitig terminieren.
https://github.com/ProjektAdLer/Autorentool/commit/d168a496facd8015500c069b6a6f4efb40f7a1a9
Problem tritt auch in v1.0 auf, fix bitte auf den Support Branch backporten
Beschreibung des Problems:
Bei gesetztem BackendToken aber leerer BackendBaseUrl in ApplicationConfig stürzt das Programm bei Öffnen des LmsLoginDialog ab.
Schritte zum Reproduzieren:
Leere BackendBaseUrl und beliebigen String in BackendToken in ApplicationConfig.json abspeichern, Autorentool starten, LmsLoginDialog öffnen.
Erwartetes Verhalten:
Es wird nicht versucht, den Token über das Backend zu verifizieren, sondern der Token wird einfach gecleared.
Tatsächliches Verhalten:
Wir versuchen einen Request an eine leere URI zu senden um den Token zu verifizieren.
Ansprechpartner:
@andreasweishaupt @niklasstich @philgei
Ursache (erst bei Fix angeben):
Kein Check in BusinessLogic.UpdateUserInformation ob BackendBaseUrl gesetzt ist.
Lösung (erst bei Fix angeben):
Check ob BackendBaseUrl gesetzt ist, wenn nicht Logout() und frühzeitig terminieren.
Commit, in dem das Problem gefixed wurde:
https://github.com/ProjektAdLer/Autorentool/commit/d168a496facd8015500c069b6a6f4efb40f7a1a9
Notes
Problem tritt auch in v1.0 auf, fix bitte auf den Support Branch backporten