Skip to content

Commit

Permalink
feature #22708 Adding autowire alias for AuthenticationUtils (weaverr…
Browse files Browse the repository at this point in the history
…yan)

This PR was merged into the 3.3-dev branch.

Discussion
----------

Adding autowire alias for AuthenticationUtils

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | none
| License       | MIT
| Doc PR        | none

AuthenticationUtils is a public service (used commonly in your `loginAction`), so it should be autowireable.

We might need to do a full audio of public services to add aliases. Since we're moving in the direction of type-based autowiring, these aliases are sort of the new "public" - it's the list of things you're "supposed" to use.

Commits
-------

fe40cb2 Adding autowire alias for AuthenticationUtils
  • Loading branch information
fabpot committed May 13, 2017
2 parents 50729d5 + fe40cb2 commit 489fd07
Showing 1 changed file with 1 addition and 0 deletions.
Expand Up @@ -68,6 +68,7 @@
<service id="security.authentication_utils" class="Symfony\Component\Security\Http\Authentication\AuthenticationUtils" public="true">
<argument type="service" id="request_stack" />
</service>
<service id="Symfony\Component\Security\Http\Authentication\AuthenticationUtils" alias="security.authentication_utils" />

<!-- Authorization related services -->
<service id="security.access.decision_manager" class="Symfony\Component\Security\Core\Authorization\AccessDecisionManager">
Expand Down

0 comments on commit 489fd07

Please sign in to comment.