auth: Make 'LocalUserCreated' hook non-abortable
authorTimo Tijhof <krinklemail@gmail.com>
Fri, 29 Mar 2019 18:41:56 +0000 (18:41 +0000)
committerTimo Tijhof <krinklemail@gmail.com>
Fri, 29 Mar 2019 18:41:56 +0000 (18:41 +0000)
commitef3b8bf08bd75b136f4a6774856cecd1dc9a2d91
tree1d498d64963e47c11e0b7222f054c2136ae917a1
parentd159839c392dcd1dc1373b774136f5f9746e51f8
auth: Make 'LocalUserCreated' hook non-abortable

Its return value was already not used. By using runWithoutAbort()
it means that if a hook accidentally returns false, it will throw
an exception instead of silently skipping other hook handlers.

This type of mistake has repeatedly caused problems in WMF prod
over the years.

Change-Id: I6af2261957eb20f18fa5f0d50c4c4835d3850cfe
includes/auth/AuthManager.php