coppermine-gallery.com/forum

Dev Board => cpg1.4 Testing/Bugs => cpg1.4 Testing/Bugs: FIXED/CLOSED => Topic started by: rollabones on August 16, 2007, 02:06:26 AM

Title: forgot password multiple emails
Post by: rollabones on August 16, 2007, 02:06:26 AM
I'm just now setting up and testing 1.4.12. I just tested the forgot password process.

- I clicked "I forgot my password" on login page, entered my email address, and clicked the go button.
- I received an email that said to click on link to have new password emailed to you, so I clicked link.
- This opened a new web browser window, which said it was sending me an email. I then received 3 different emails, each with a different new password.

Steve
http://gallery.rollanet.org/
Title: Re: forgot password multiple emails
Post by: Nibbler on August 16, 2007, 04:53:46 AM
Not possible. The forgot password page only sends one email per request.
Title: Re: forgot password multiple emails
Post by: rollabones on August 18, 2007, 04:24:05 PM

I tried it twice on two different computers.

The email is a Yahoo email address. When I click link that says I do want a new password emailed to me, another web browser window opens and I receive more than one email each with a different password. On my Windows 2000 PC I received 3 emails and on my Windows XP PC I received 2 emails.

I changed the email address on my account to another email address which I read using Pegasus Mail. The link was not active in that email, so I copied and pasted it into a web browser and submitted it. I received 2 emails each with a different password.

Steve

Title: Re: forgot password multiple emails
Post by: Nibbler on August 18, 2007, 04:59:41 PM
Try removing the google ads from the forgot password page.
Title: Re: forgot password multiple emails
Post by: rollabones on August 18, 2007, 05:17:16 PM
I'm not quite sure how to do that, since the Google Ads are in template.html.
Title: Re: forgot password multiple emails
Post by: Nibbler on August 18, 2007, 05:24:49 PM
Change


    if (!$result) {
        cpg_die($lang_forgot_passwd_php['forgot_passwd'], $lang_forgot_passwd_php['illegal_session']);
    }


to


    if (!mysql_num_rows($result)) {
        cpg_die($lang_forgot_passwd_php['forgot_passwd'], $lang_forgot_passwd_php['illegal_session']);
    }


Please report back if the fix is correct.
Title: Re: forgot password multiple emails
Post by: rollabones on August 18, 2007, 05:46:28 PM
Yes! That fixed it.

Thanks!!!