Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.2k
[Security] document current user attribute in the main security doc entry#20864
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:7.2
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
| @@ -2100,6 +2101,11 @@ accessed via the ``getUser()`` shortcut in the | |||
| // For example, if you added a getFirstName() method, you can use that. | |||
| return new Response('Well hi there '.$user->getFirstName()); | |||
| } | |||
| public function me(#[CurrentUser] User $user): Response | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
What about unauthenticated users?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
good question, should I add an IsGranted above the action? :)
Uh oh!
There was an error while loading.Please reload this page.
I know its documented rapidly (via a code diff example) on top of this file
but here on a controller I thinks its more usefull rather than using the getUser of the abstract controller
wdyt?
also unrelated but, would it be possible to display the class name here ?
seeing only
getUsercan be misleading, what aboutSecurity::getUserbecause getUser is globally generic in the codebase