• Resolved kastoberlin

    (@kastoberlin)


    Starting yesterday after upgrading to the new V.2.1.3 version every user receives a password change email after successful login.

    I verified that this is caused by your plugin because when I either deactivate the plugin comletely, deactivate NADI within the plugin, or uncheck the option to automatically sync the password on each login, these emails are not sent. AD authentification continues to work fine though.

    We want local passwords to be synchronized on each login in order to have a fallback in case LDAP is not accessible and this was not a problem in the previous version

    Please advise

Viewing 5 replies - 1 through 5 (of 5 total)
  • I was just getting ready to post about this as well. I fixed this on my site by adjusting next-active-directory-integration/classes/Adi/User/Manager.php lines 357-358 to be as follows:

    Original:

    $this->logger->debug('Setting local password to the one used for this login.');
    $this->userRepository->updatePassword($userId, $password);

    New:

    $this->logger->debug('Setting local password to the one used for this login.');
    $this->disableEmailNotification();
    $this->userRepository->updatePassword($userId, $password);
    • This reply was modified 6 years, 2 months ago by ericjuden. Reason: Adding code tags around new code block
    Thread Starter kastoberlin

    (@kastoberlin)

    Hi Eric,
    thanks for this tip. Usually I am hesitant to modify plugin code directly, but this patch looks straightforward and it seems to work. Thanks again!

    Agreed @kastoberlin, I’m not big on changing plugin files either. In the interim until they have it fixed, it’ll have to work. I just figured I’d pass it along in case you had to get it fixed as well.

    Hello @kastoberlin and @ericjuden,

    we are aware of this bug and will fix it the way @ericjuden described in his previous answer.

    We will release the fix in the next NADI version.

    Best regards,
    medan123

    Fixed with NADI 2.1.4

    Best regards,
    medan123

Viewing 5 replies - 1 through 5 (of 5 total)
  • The topic ‘Password Change Emails are sent on every login’ is closed to new replies.