Commit ce612ac
committed
minor #11459 Fixed additional '@' in DoctrineTokenProvider service in 'Remember Me' functionality (vertisan)
This PR was submitted for the 4.2 branch but it was merged into the 3.4 branch instead (closes #11459).
Discussion
----------
Fixed additional '@' in DoctrineTokenProvider service in 'Remember Me' functionality
In current version we have to add the following line into our firewall definitions:
```yaml
token_provider: '@symfony\Bridge\Doctrine\Security\RememberMe\DoctrineTokenProvider'
```
but with this we'll get error:
```
The service "security.authentication.rememberme.services.persistent.main" has a dependency on a non-existent service "@symfony\Bridge\Doctrine\Security\RememberMe\DoctrineTokenProvider".
```
To fix this we need to remove `@` character from service definition so finally it should looks like:
```yaml
token_provider: 'Symfony\Bridge\Doctrine\Security\RememberMe\DoctrineTokenProvider'
```
P.S. I have already commit for `master` branch too so if this change will be accepted, I'll create pull request for `master` too.
Commits
-------
9e137fd Fixed additional '@' in DoctrineTokenProvider service in 'Remember Me' functionality1 file changed
+3
-3
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
329 | 329 | | |
330 | 330 | | |
331 | 331 | | |
332 | | - | |
| 332 | + | |
333 | 333 | | |
334 | 334 | | |
335 | 335 | | |
| |||
348 | 348 | | |
349 | 349 | | |
350 | 350 | | |
351 | | - | |
| 351 | + | |
352 | 352 | | |
353 | 353 | | |
354 | 354 | | |
| |||
365 | 365 | | |
366 | 366 | | |
367 | 367 | | |
368 | | - | |
| 368 | + | |
369 | 369 | | |
370 | 370 | | |
371 | 371 | | |
| |||
0 commit comments