vova07 / yii2-users-module

This module provide a users managing system for your yii2 application.
24 stars 15 forks source link

Email error - recovery email #2

Closed em17 closed 10 years ago

em17 commented 10 years ago

When i click the recover button in the password recovery, it gives this error "Maximum execution time of 30 seconds exceeded"

my mail config is:

'mail' => [ 'class' => 'yii\swiftmailer\Mailer', 'useFileTransport' => false, 'transport' => [ 'class' => 'Swift_SmtpTransport', 'host' => 'smtp.gmail.com', 'username' => 'example@gmail.com', 'password' => 'password', 'port' => '465', 'encryption' => 'tls', ], ],

vova07 commented 10 years ago

Can you check if your mail component work? Try to send some emails via swiftmailer.

em17 commented 10 years ago

Yes, sure I will try again and come back to you with the results.. Also it would be great if you can make an email address of yours available in your git profile so for minor issues we can email you, instead of opening issues. I really like this module by the way.

vova07 commented 10 years ago

In composer.json are indicated my contacts. But better practice is to use issues, so I'll never forget about the problem.

em17 commented 10 years ago

Thanks for your email.

I found that in my mail config this line 'encryption' => 'tls' should be 'encryption' => 'ssl' now I don't get the timeout error but no email is sent to the user's email address...

anywhere else I need to configure..?

em17 commented 10 years ago

Finally I got an password recovery email but this is the URL it doesn't have any auth key

'http://localhost/vova/web/index.php?r=users%2Fguest%2Frecovery'

vova07 commented 10 years ago

I fixed the problem. Run composer update for users module, and try again.

em17 commented 10 years ago

Thank you for the recovery key fix! I checked it, it has the recovery key now but there's another little problem it doesn't have the domain name this is what I got '/vova/web/index.php?r=users%2Fguest%2Frecovery&key=XXXEXAMPLEXXXXX' This part is missing 'http://localhost'

vova07 commented 10 years ago

Sorry, my mistake. Fixed it.

em17 commented 10 years ago

There's another little fix needed sorry I didn't notice that earlier.

URL should route to 'recovery-confirmation' NOT again to 'recovery'

vova07 commented 10 years ago

Yes you're right. Thanx!

em17 commented 10 years ago

Great! It's working, thank you!