Changes between Initial Version and Version 1 of Ticket #5352, comment 5


Ignore:
Timestamp:
Apr 25, 2018, 10:06:34 AM (6 years ago)
Author:
Olivier Mehani

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #5352, comment 5

    initial v1  
    11It sounds like your patches would work for me yeah.
    22
    3 That said, the underlying reason why I think `allow_registration = true` is for the case where you have more than one authmethod enabled (if that's even possible), where you have an LDAP directory of known users, as well as a local base of local users independent from LDAP. You may want users from LDAP to have local users created seamlessly when first login (I'm happy with the email verification), but not let non-LDAP non-local users create new accounts on the fly.
     3That said, the underlying reason why I think ignoring `allow_registration = false` is for the case where you have more than one authmethod enabled (if that's even possible), where you have an LDAP directory of known users, as well as a local base of local users independent from LDAP. You may want users from LDAP to have local users created seamlessly when first login (I'm happy with the email verification), but not let non-LDAP non-local users create new accounts on the fly.
    44
    55Spelled out like that, I agree it's a bit contrived.