Bug #80389

Login attempt not marked as failed when having an anonymous session

Added by Mads Lønne Jensen over 2 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Frontend
Target version:
Start date:
2017-03-22
Due date:
% Done:

100%

TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Is Regression:
Yes
Sprint Focus:
Stabilization Sprint

Description

In AbstractUserAuthentication#checkAutentication the $anonymousSession indicates whether the current user session is anonymous.
When it is set, login attempts are not marked as failed in AbstractUserAuthentication->loginFailure

Associated revisions

Revision 3ae3ffb8 (diff)
Added by Mads Lønne Jensen over 2 years ago

[BUGFIX] Mark login attempts as failed for anonymous sessions

Fixes a bug where login attempts would not be marked with loginFailure
when an existing anonymous session exists.

Resolves: #80389
Releases: master
Change-Id: Id421d95fd5239c7a16e771463db3d9eb372d0340
Reviewed-on: https://review.typo3.org/52116
Tested-by: TYPO3com <>
Reviewed-by: Thomas Hohn <>
Reviewed-by: Morten Pless <>
Reviewed-by: Anders Kostending <>
Reviewed-by: Faton Haliti <>
Reviewed-by: Markus Klein <>
Tested-by: Markus Klein <>
Reviewed-by: Wouter Wolters <>
Reviewed-by: Frank Nägler <>
Tested-by: Frank Nägler <>

History

#1 Updated by Gerrit Code Review over 2 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52116

#2 Updated by Gerrit Code Review over 2 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52116

#3 Updated by Gerrit Code Review over 2 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/52116

#4 Updated by Markus Klein over 2 years ago

  • Category set to Frontend
  • Priority changed from Should have to Must have
  • Target version set to 8 LTS
  • Is Regression changed from No to Yes
  • Sprint Focus set to Stabilization Sprint

#5 Updated by Mads Lønne Jensen over 2 years ago

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

#6 Updated by Riccardo De Contardi almost 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF