Unable to reset password following a forgotten password request

[gazzastone]

Version
1.3.9

Steps To reproduce
Click on forgotten password email link and complete Reset Password dialog.

Expected Result
Expect password to be changed.

Actual result
ErrorException thrown with message "strtotime() expects parameter 1 to be string, array given".

Stacktrace:

#25 ErrorException in C:\xampp\htdocs\learninglocker\vendor\laravel\framework\src\Illuminate\Auth\Reminders\DatabaseReminderRepository.php:109

[ryasmi]

@gazzastone this is a good one. I'm literally clueless. It looks like you have managed to create a user without a created_at property which is out of our control.

[ryasmi]

Note to self:
https://github.com/jenssegers/Laravel-MongoDB#auth
mongodb/laravel-mongodb#37

Adding 'Jenssegers\Mongodb\Auth\PasswordResetServiceProvider' to app.php didn't help.

[ryasmi]

I can't reproduce the issue, but when I try to fix it everything breaks.

[gazzastone]

ok I will delete this user then recreate it and then do a password reset to see if the issue has gone

[ryasmi]

Ok sounds good.

[gazzastone]

so deleted user. registered user ok. requested password reset get the same error

[ryasmi]

No way! That's crazy, I'll see what I can find out. We're all pretty busy over here today, so it may have to wait a little while.

[gazzastone]

ok no probs

[sanduhrs]

Same issue here, created_at actually is an array at least in my setup at 00e3a01

created_at = {array}[3]
  date = "2015-04-22 13:11:06.000000"
  timezone_type = 3
  timezone = "UTC"

[sanduhrs]

Using the instructions from #163 fixes the issue for me.
Should this be documented at http://docs.learninglocker.net/installation/ or is it a bug?

[gazzastone]

I can confirm that #163 has fixed the issue for me many thanks @sanduhrs

[ryasmi]

Ohhhhhh I'm a complete idiot! My apologies guys! I was well aware of this, but incorrectly assumed you were using the local configs.

@sanduhrs I think what needs to be documented is that you should always use the local config or duplicate the local config and use your own. Using your own just requires you to change the "bootstrap/start.php" file a little.

[ryasmi]

This explains why I couldn't reproduce this issue. We can't add that service provider in a commit because it screws up our continuous integration. However, we could potentially change "bootstrap/start.php" so that it contains...

$env = Helpers::getEnvironment([
  'local' => [gethostname()]
], gethostname());

This would ensure that you always used the local config unless you changed this file, which seems ideal to me.

[gazzastone]

sounds like a plan to me

[ryasmi]

This issue has now been fixed and the fix has been merged into the develop branch. This should be released with version 1.3.10. Thanks @gazzastone and @sanduhrs for all of your help here.