Remove needless ContainerAware dependency from AuthenticationManager

... In fact we just store the service ID to not have to figure out which authentication manager applies to which route at runtime. I would love to extend #2213319: Create a single Container CompilerPass to collect + add handlers to consumer service definitions to support service IDs as well.

This time it should actually work.

Fixed the failure.

A welcome improvement.

this shouldn't hold the commit up but

$id = substr($id, strlen('authentication.'));

that just rubs me the wrong way. Is there any reason not to put that in a comment and instead run a strlen every time this is ran? I know microoptimization and all but still, this is ridiculous (and if I am not mistaken this runs on every request). Do we expect that the length of the string authentication. to change any time soon?

+1 for this optimization. This runs at least on potentially every request, so its somehow worth.

Committed a704796 and pushed to 8.0.x. Thanks!