Details des Tickets
Beschreibung,Kommentare und Anhänge
| merge request id | 538 |
| Titel | LL-1035 introduced password expiration days in client, for clients this field ist… |
| Beschreibung | + introduced password expiration days in client, for clients this field ist relevant and not general config
Bitte im Rahmen dieses Task zusammen mit Sascha Immig nochmal den Status Quo prüfen.
Ziel soll es sein, dass der aktuell in der Plattform hinterlegte Zeitraum für das Ablaufen von Passwörtern (365Tage) die Standard-Einstellung für Clients ist. Der Client-Admin soll in den Mandanteneinstellungen die Dauer anpassen und aussetzen können.
Zusätzlich soll weiterhin die Option bestehen bleiben, bei aktiviertem Auslaufen der Passwörter, einzelne Nutzer davon ausnehmen u können. (D.h.: wurde das Ablaufen clientweit deaktiviert, sollten die Radio-Buttons für die Nutzerspezifische Einstellung auch deaktiviert/ausgeblendet werden.)
Direkt ein Szenario dass sich daraus ergibt:
Kunde hat Ablaufen aktiv und nimmt einen User davon aus. Danach stellt er den gesamten Mandanten um. Auch wenn die Radios deaktiviert/ausgeblendet sind, werden die Infos am User nicht gelöscht. Reaktiviert der Kunde nun das Ablaufen wieder (Auf Mandaten-Ebene), werden alle MAs, deren Passwort alt genug ist, zur Neuvergabe gebeten. Der Mitarbeiter, der vorher ausgenommen wurde, wäre nun wieder / weiterhin ausgenommen und müsste sein Passwort nicht neu setzen. |
| Status | merged |
| Created at | 2021-12-09T09:05:47.137Z |
| Updated at | 2022-01-12T08:29:43.542Z |
| Source branch | feature/LL-1035-client-individual-password-expiration-config |
| Target branch | develop |
| Author | Gregor Gabriel |
| Assignee | Sascha Immig |
| würde das nicht kürzer gehen (und übersichtlicher):
`@if(config('security.passwordExpirationDays') && (! $client->id || $client->password_expiration_days))`
Nach der Prüfung der Config sollte `!$client->id` doch für den Part hinter dem OR die ClientID implizieren, oder? |
Sascha Immig |
2021-12-13T07:11:52.879Z |
| stehen die PasswortExpirationDays in der initialen Config (`.env.example`)? |
Sascha Immig |
2021-12-13T07:13:30.505Z |
| Bitte Klammern.
Macht es Sinn, wie oben zuerst die Config abzufragen? Ist das Global deaktiviert, muss ich den Rest nicht machen. |
Sascha Immig |
2021-12-13T07:15:39.289Z |
| Geht leider nicht, denn wenn config('security.passwordExpirationDays') den Wert NULL hat,
dann kannst du im Mandanten einstellen was du willst es wird nicht angewendet |
Gregor Gabriel |
2021-12-13T07:33:43.030Z |
| ja steht drin, aber auskommentiert |
Gregor Gabriel |
2021-12-13T07:35:37.670Z |
| Die allgemeine Config spielt nur noch beim Anlegen des Mandanten eine Rolle, oder wenn der Mandant das Feature deaktiviert und dann wieder aktiviert (dann wird im UI bei der Reaktivierung der default Wert aus der Config genommen). Ansonsten wird immer der Wert aus dem Mandanten genommen. |
Gregor Gabriel |
2021-12-13T07:38:58.229Z |
| changed title from **introduced password expiration days in client, for clients this field ist…** to **{+LL-1035 +}introduced password expiration days in client, for clients this field ist…** |
Sascha Immig |
2021-12-15T20:47:18.045Z |
| changed the description |
Sascha Immig |
2021-12-15T20:47:18.075Z |
| was schreibt das Skript dann in die DB, wenn der Wert in der .env auskommentiert ist? |
Sascha Immig |
2021-12-15T20:51:34.419Z |
| Können wir also nicht mehr plattformweit das Feature deaktivieren? |
Sascha Immig |
2021-12-15T20:52:44.176Z |
| Ich würde davon ausgehen, dass wenn die Systemkonfiguration das Feature abstellt (also NULL drin steht), dass dann die Funktion für Mandanten nicht zur Verfügung steht.
Vor der Anpassung der mandantenspezifischen Ablauf-Zeit müsste das doch auch schon so gewesen sein - also keine Einstellung am User möglich... |
Sascha Immig |
2021-12-15T20:54:32.605Z |
| Nach Rücksprache: Bleibt logisch so wie es jetzt ist. Bitte aber klammern! |
Sascha Immig |
2021-12-16T10:04:49.264Z |
| Nach Rücksprache: Bleibt logisch so wie es jetzt ist. Bitte aber klammern! |
Sascha Immig |
2021-12-16T10:04:56.763Z |
| Default-Wert systemisch ist 365. |
Sascha Immig |
2021-12-16T10:42:20.901Z |
| changed this line in version 2 of the diff |
Gregor Gabriel |
2021-12-16T11:05:00.746Z |
| changed this line in version 2 of the diff |
Gregor Gabriel |
2021-12-16T11:05:01.251Z |
added 2 commits
- f5a48ae0 - changed default
- 6b45ccd6 - adden brackets to code and did small restructuring of if code
Compare with previous version |
Gregor Gabriel |
2021-12-16T11:05:01.396Z |
| Habe den Code geklammert und umgebrochen |
Gregor Gabriel |
2021-12-16T11:10:32.257Z |
| default 365 gesetzt, wenn die Konstante nicht gesetzt ist |
Gregor Gabriel |
2021-12-16T11:11:13.736Z |
| resolved all discussions |
Gregor Gabriel |
2021-12-16T11:11:18.284Z |
| Habe den Code geklammert und umgebrochen |
Gregor Gabriel |
2021-12-16T11:11:27.316Z |
| resolved all discussions |
Gregor Gabriel |
2021-12-16T11:11:29.077Z |
| resolved all discussions |
Gregor Gabriel |
2021-12-16T11:23:55.591Z |
added 16 commits
- 6b45ccd6...34fbbcb3 - 15 commits from branch
develop - 57396576 - Merge branch 'develop' of https://gitlab.learningsystem.de/lms/seventeen into…
Compare with previous version |
Gregor Gabriel |
2021-12-16T11:23:55.892Z |
| merged |
Sascha Immig |
2022-01-12T08:29:43.619Z |
| mentioned in commit 45be6431e128b1f805f80bbdd8c9fc1c7d32e7e3 |
Sascha Immig |
2022-01-12T08:29:43.755Z |
| Title |
Id |
Author |
Created at |
Message |
| Merge branch 'develop' of https://gitlab.learningsystem.de/lms/seventeen into… |
573965767c7478b678aead3793a90718b6ca0076 |
TUV\gabrielg |
2021-12-16T11:23:36.000Z |
Merge branch 'develop' of https://gitlab.learningsystem.de/lms/seventeen into feature/LL-1035-client-individual-password-expiration-config
|
| adden brackets to code and did small restructuring of if code |
6b45ccd60bcbf4b570a11e8ba4e251b345b7842b |
TUV\gabrielg |
2021-12-16T11:04:22.000Z |
adden brackets to code and did small restructuring of if code
|
| + improved ui, so if password expiration is deactivated this is displayed by a deactivated checkbox |
83907d0d15fb1f76e7c6d25f4b525aa297813978 |
TUV\gabrielg |
2021-12-09T09:04:07.000Z |
+ improved ui, so if password expiration is deactivated this is displayed by a deactivated checkbox
|
| + introduced password expiration days in client, for clients this field ist… |
a726948c77be7f26c6a2c9de94dec0543e9a3b53 |
TUV\gabrielg |
2021-12-08T15:25:11.000Z |
+ introduced password expiration days in client, for clients this field ist relevant and not general config
|
| missing part for ui |
e90abd52fb32852088bed94391d41232c2a00c64 |
TUV\gabrielg |
2021-12-09T09:04:48.000Z |
missing part for ui
|
| changed default |
f5a48ae09d84f3f8f86fd48f48b97cdcaae8c305 |
TUV\gabrielg |
2021-12-16T09:48:50.000Z |
changed default
|