Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

As service admin I want to view the last login date for a user in LDAP #30

Closed
baszoetekouw opened this issue May 3, 2022 · 9 comments · Fixed by #130
Closed

As service admin I want to view the last login date for a user in LDAP #30

baszoetekouw opened this issue May 3, 2022 · 9 comments · Fixed by #130
Milestone

Comments

@baszoetekouw
Copy link
Member

baszoetekouw commented May 3, 2022

Belangrijkste vraag: in welk attribuut willen we dit stoppen?

@baszoetekouw baszoetekouw moved this to Icebox in SRAM development May 3, 2022
@baszoetekouw baszoetekouw self-assigned this May 3, 2022
@baszoetekouw baszoetekouw moved this from Icebox to Todo in SRAM development May 3, 2022
@FlorisFokkinga FlorisFokkinga changed the title Als diensteigenaar wil ik in de ldap de datum van de laatste login van de gebruiker kunnen zien. As service admin I want to view the last login date for a user in LDAP May 3, 2022
@baszoetekouw baszoetekouw moved this from Todo to Icebox in SRAM development May 10, 2022
@baszoetekouw baszoetekouw moved this to In progress in SRAM development May 17, 2022
@baszoetekouw baszoetekouw moved this from In progress to Todo in SRAM development May 17, 2022
@baszoetekouw baszoetekouw added this to the v15 milestone May 17, 2022
@FlorisFokkinga
Copy link

We are now convinced we need a custom schema anyway, so this now is a good candidate for a first addition.

@baszoetekouw baszoetekouw removed this from the v15 milestone Aug 16, 2022
@baszoetekouw
Copy link
Member Author

Next step: maak een ontwerp voor dat custom ldap schema dat we willen gaan gebruiken

@baszoetekouw
Copy link
Member Author

Zie SURFscz/SRAM-deploy#498

@baszoetekouw baszoetekouw assigned mrvanes and unassigned baszoetekouw Feb 28, 2024
@mrvanes mrvanes moved this from Todo to In progress in SRAM development Mar 1, 2024
@mrvanes
Copy link
Contributor

mrvanes commented Mar 20, 2024

Wat worden de brongegevens van SBS om deze te vullen?

@baszoetekouw
Copy link
Member Author

SBS heeft (sinds kort denk ik) users.[n].last_login_date in de plsc/sync

@baszoetekouw
Copy link
Member Author

Oh, en we moeten dit een klein beetje privacy-vriendelijk maken denk ik en niet de precieze datetime van de laatste login opnemen, maar een wat grovere afronding. Ik denk dat afronden op een week wel ok is; wat denkt @arnoutt ?

@baszoetekouw
Copy link
Member Author

baszoetekouw commented Mar 20, 2024

Ik zit nu wel te denken of het dan niet handiger is om in de ldap een sramDaysInactive op te nemen ipv sramLastActivityDate. Dat rond ook makkelijker af.

@baszoetekouw
Copy link
Member Author

Het ei is gelegd: we willen de volgende intervallen gebruiken

  • <= 5 dagen: resolutie 1 dag
  • <= 4 weken: resolutie 1 week
  • <= 12 maanden: resolutie 1 maand
  • daarna: resolutie 3 maanden (maar zou niet voor moeten komen)

En dat dan in een integer-waarde sramInactiveDays stoppen in de ldap (dus het schema uit SURFscz/SRAM-deploy#498) moet dan ook wel worden uitgebreid

@mrvanes mrvanes linked a pull request Mar 21, 2024 that will close this issue
@mrvanes mrvanes moved this from In progress to To be tested in SRAM development Mar 21, 2024
@baszoetekouw baszoetekouw added this to the v33 milestone Apr 8, 2024
@baszoetekouw baszoetekouw moved this from To be tested to Being tested in SRAM development Apr 9, 2024
@baszoetekouw baszoetekouw moved this from Being tested to To be tested in SRAM development May 7, 2024
@baszoetekouw baszoetekouw modified the milestones: v33, v34 May 7, 2024
@baszoetekouw
Copy link
Member Author

baszoetekouw commented Jun 10, 2024

Tested, works also on prod data (plsc run take a long time though)
Also added to docs at https://servicedesk.surf.nl/wiki/display/WIKI/LDAP+directory+structure

@baszoetekouw baszoetekouw moved this from To be tested to To be deployed in SRAM development Jun 10, 2024
@sram-deploy-tools-automation sram-deploy-tools-automation bot moved this from To be deployed to Done in SRAM development Aug 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants