First login event #26206
First login event #26206
Conversation
|
@DeepDiver1975, thanks for your PR! By analyzing the annotation information on this pull request, we identified @ChristophWurst, @blizzz, @icewind1991 and @DeepDiver1975 to be potential reviewers |
DeepDiver1975
force-pushed
the
first-login-event
branch
2 times, most recently
from
43d9b75
to
899d6e7
Compare
…operly be processed for the admin
DeepDiver1975
force-pushed
the
first-login-event
branch
from
899d6e7
to
a68d631
Compare
| Util::connectHook('OC_User', | ||
| 'post_login', | ||
| $this, | ||
| 'postLogin'); |
There was a problem hiding this comment.
I'm not sure we can get rid of this. IIRC some auth code and ext storage relies on this
There was a problem hiding this comment.
This is only the listener within dav.
| @@ -349,7 +349,6 @@ public function getUserFolder($userId) { | |||
| $folder = $folder->get($dir); | |||
| } catch (NotFoundException $e) { | |||
| $folder = $folder->newFolder($dir); | |||
| \OC_Util::copySkeleton($userId, $folder); | |||
| //trigger creation of user home and /files folder | ||
| \OC::$server->getUserFolder($user); | ||
|
|
||
| if ($firstTimeLogin) { |
There was a problem hiding this comment.
I suppose we can also leverage this to avoid syncing the whole LDAP profile every time a LDAP user logs in ? (something to discuss separately)
CC @butonic
|
Tested with local users and LDAP, works 👍 Tested by adding an event handler in files_external's app and logging the user from the event. |
|
added to features wiki page: https://github.com/owncloud/core/wiki/ownCloud-9.2-Features |
|
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Description
Introduces the first login event
Related Issue
Motivation and Context
In the past we had to miss-use the login hook to run initialization code.
Now there is an isolated event
Types of changes
Checklist: