Skip to content
Snippets Groups Projects

fixes #40 by recalculating groups _before_ to_ldap(), which sets the groups in ldap

fixes #40 by recalculating groups _before_ to_ldap(), which sets the groups in ldap
All threads resolved!

recalculate_groups (or the expanded version of that) was called after user creation in LDAP, but the user was not updated afterwards to actually set the groups.

running recalculate_groups before .to_ldap() sets the groups right to only need one .to_ldap().

Note: recalculate_groups only works on the Role/User, so we don't need to wait for LDAP success. .to_ldap() first adds the user and only afterwards does the groups.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • nd approved this merge request

    approved this merge request

  • sistason added 1 commit

    added 1 commit

    • 049d1f3c - removed obsolete debug-print

    Compare with previous version

  • sistason resolved all threads

    resolved all threads

  • sistason enabled an automatic merge when the pipeline for 049d1f3c succeeds

    enabled an automatic merge when the pipeline for 049d1f3c succeeds

  • merged

  • Please register or sign in to reply
    Loading