I would keep things "as is" for backward compatibility. When a precise analysis is required, the ```getSuccesses()``` method needs to be used : it will certainly be the case for LOA.
At a code level, I think it can be good to isolate this loop into a new separate method : ```saveHandlerNameInAuthentication``` (you can find a better name) which can be overriden into another manager.