Feature #29698

New Login error hook

Added by Alessandro Bellafronte almost 8 years ago. Updated 11 months ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
felogin
Target version:
-
Start date:
2011-09-12
Due date:
% Done:

100%

PHP Version:
Tags:
Complexity:
easy
Sprint Focus:

Description

I noticed that there is no "login-error" hook in felogin. Such a hook would be useful to implement security enhancements.

I saw that a similar hook exists yet a couple of lines before ("login confirmed"-hook) so I suggest to implement a login error hook too.

Appended to this issue you will find a patch-file which implements the requested hook.

Kind regards
Alessandro Bellafronte

class.tx_felogin_pi1.php.patch View - Patch File (960 Bytes) Alessandro Bellafronte, 2011-09-12 17:57


Related issues

Related to TYPO3 Core - Feature #33717: New Login error hook Closed 2012-02-06

Associated revisions

Revision aafcdf26 (diff)
Added by Simon Schaufelberger about 7 years ago

[FEATURE] New login error hook

Add new login hook to handle login errors.

Change-Id: I72b91b78955e27e4123f458e4bf6c50ab150397b
Resolves: #29698
Releases: 6.0
Reviewed-on: http://review.typo3.org/6600
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert

Revision 798cf0e2 (diff)
Added by Simon Schaufelberger about 7 years ago

[FEATURE] New login error hook

Add new login hook to handle login errors.

Change-Id: I72b91b78955e27e4123f458e4bf6c50ab150397b
Resolves: #29698
Releases: 6.0
Reviewed-on: http://review.typo3.org/6600
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert

Revision e2893d80 (diff)
Added by Stephan Großberndt 4 months ago

[TASK] Add missing fe_login hooks to docs

Add the missing documentation of the hooks

  • `password_changed` added in TYPO3 4.3
  • `login_error` added in TYPO3 6.0

Releases: master, 9.5, 8.7
Resolves: #88131
Related: #87726, #29698
Change-Id: I0dc875a399da58e13b15225e173392565c64bb03
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/60443
Tested-by: TYPO3com <>
Tested-by: Julian Geils <>
Tested-by: Steffen Frese <>
Tested-by: Tobi Kretschmann <>
Reviewed-by: Julian Geils <>
Reviewed-by: Steffen Frese <>
Reviewed-by: Tobi Kretschmann <>

Revision 676d5f72 (diff)
Added by Stephan Großberndt 4 months ago

[TASK] Add missing fe_login hooks to docs

Add the missing documentation of the hooks

  • `password_changed` added in TYPO3 4.3
  • `login_error` added in TYPO3 6.0

Releases: master, 9.5, 8.7
Resolves: #88131
Related: #87726, #29698
Change-Id: I0dc875a399da58e13b15225e173392565c64bb03
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/60486
Tested-by: TYPO3com <>
Tested-by: Tobi Kretschmann <>
Reviewed-by: Tobi Kretschmann <>

Revision cf623c4e (diff)
Added by Stephan Großberndt 4 months ago

[TASK] Add missing fe_login hooks to docs

Add the missing documentation of the hooks

  • `password_changed` added in TYPO3 4.3
  • `login_error` added in TYPO3 6.0

Releases: master, 9.5, 8.7
Resolves: #88131
Related: #87726, #29698
Change-Id: I0dc875a399da58e13b15225e173392565c64bb03
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/60527
Tested-by: Stephan Großberndt <>
Tested-by: Tobi Kretschmann <>
Tested-by: TYPO3com <>
Reviewed-by: Stephan Großberndt <>
Reviewed-by: Tobi Kretschmann <>

History

#1 Updated by Tobias Liebig almost 8 years ago

Seems to be a really easy task. Thanks for contributing the patch.

Would you mind to push it as a change request in Gerrit?

You're not used to Git/Gerrit? No problem: check http://wiki.typo3.org/Git

#2 Updated by Mr. Hudson almost 8 years ago

  • Status changed from New to Under Review

Patch set 1 of change I72b91b78955e27e4123f458e4bf6c50ab150397b has been pushed to the review server.
It is available at http://review.typo3.org/6600

#3 Updated by Mr. Hudson almost 8 years ago

Patch set 2 of change I72b91b78955e27e4123f458e4bf6c50ab150397b has been pushed to the review server.
It is available at http://review.typo3.org/6600

#4 Updated by Mr. Hudson almost 8 years ago

Patch set 3 of change I72b91b78955e27e4123f458e4bf6c50ab150397b has been pushed to the review server.
It is available at http://review.typo3.org/6600

#5 Updated by Gerrit Code Review over 7 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/6600

#6 Updated by Gerrit Code Review over 7 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/6600

#7 Updated by Gerrit Code Review over 7 years ago

Patch set 6 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/6600

#8 Updated by Gerrit Code Review over 7 years ago

Patch set 7 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/6600

#9 Updated by Gerrit Code Review about 7 years ago

Patch set 8 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/6600

#10 Updated by Simon Schaufelberger about 7 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#11 Updated by Benni Mack 11 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF