RFC: Nutzernamen-Validierung erweitern

[peterkappelt]

Hallo zusammen,
der Nutzername für's Auto-Login wird gerade über diese Regex validiert:

TUfast_TUD/src/freshContent/settings/composables/logins.ts

Line 14 in d148ce6

usernamePattern: /^(s{1}\d{7}|[a-z]{4}\d{3}[a-z])$/,

Es gibt aber auch Sonderfälle, wo der ZIH-Login keine S-Nummer ist (z.B. für Mitarbeiter oder auch, wenn man einen ZIH-Login hatte, bevor man angefangen hat zu studieren). Auch bei mir, mein ZIH-Login besteht nur aus meinem Namen - und ich kann dummerweise gerade meinen Login nicht mehr hinterlegen.

Am einfachsten wäre es, die Regex einfach zu erweitern: /^(s{1}\d{7}|[a-z]{4}\d{3}[a-z])|[a-z]*$/. Ich bin mir aber nicht sicher, ob man dann noch den eigentlichen Zweck gut erfüllt (-> Leute drauf hinweisen, wenn sie z.B. ihren Namen oder Mail statt S-Nummer eingeben). Was meint ihr dazu?

[OliEfr]

Das ist ein guter Hinweis. Dieser Fall war uns noch nicht bewusst.

Welche Form hat denn dein ZIH-Login (in allgemeiner Form, aber bisschen genauer als regex)?

Ich sehe folgende Möglichkeiten:

1. Den Nutzer nur darauf hinweisen, wenn er unseren bisherigen regex nicht erfüllt (i.e. wenn er keinen Standard-ZIH-Login hat), aber speichern trotzdem zulassen.
2. Bei der alten Lösung bleiben.
3. Regex pauschal erweitern wie von @peterkappelt vorgeschlagen und alles (?) zulassen .

Ich bin für Lösung 1, da ich a) gerne allen Nutzer Zugang zu TUfast geben möchte und b) ich kein großes Problem darin sehe, wenn ein falscher Login gespeichert wird, da seit dem neuen Release der AutoLogin sowieso nur einmal ausgeführt wird.

Falls Lösung 1 zu kompliziert ist (@Noxdor ?), dann bin ich für Lösung 3.

@C0ntroller

[Noxdor]

"Lösung" 2 ist selbstverständlich keine Option, wir können keinen User ausschließen. Ich wäre für Lösung 3, für den Fall, dass @peterkappelt uns mit einer präziseren Angabe, was das Format angeht, versorgen kann. Ansonsten wäre ich auch für Lösung 1 (ist keine komplizierte Anpassung).

Danke auch nochmal fürs Feedback @peterkappelt! Somit sollten wir schnell die paar Fehlerchen ausbügeln können, die das vergleichsweise große Update gebracht hat.

[Noxdor]

Ich sehe gerade, dass ich den letzten Absatz von @peterkappelt übersprungen habe 😂 Ja, dann finde ich auch, sollten wir mit Lösung 1 gehen! Diese speziellen Loginnamen werden ja denke ich dann auch nur Leute haben, die wissen, was sie da machen.

[C0ntroller]

Ebenfalls für Lösung 1.
@Noxdor Kannst du das schnell implementieren? Ist ja nur Frontend und sollte ja eigentlich nur wenig sein, oder?

[Noxdor]

Ja, das ist schnell gemacht, sitze gleich im Zug, oder spätestens in ner Stunde in der ersten Vorlesung.

[OliEfr]

Danke dass ihr wieder sofort zur Stelle ward!
Closed by #108