Skip to content
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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make sure "pre auth" article mentions that you need an anonymous firewall #5264

Closed
weaverryan opened this issue May 17, 2015 · 1 comment
Closed

Make sure "pre auth" article mentions that you need an anonymous firewall #5264

weaverryan opened this issue May 17, 2015 · 1 comment
Labels
actionable Clear and specific issues ready for anyone to take them. good first issue Ideal for your first contribution! (some Symfony experience may be required) hasPR A Pull Request has already been submitted for this issue. Security

Comments

@weaverryan
Copy link
Member

Hi guys!

See symfony/symfony#13751 - if you're going to return null from createToken(), you must have anonymous on in your firewall so that you at least get the AnonymousToken.

Thanks!
@weaverryan weaverryan added good first issue Ideal for your first contribution! (some Symfony experience may be required) actionable Clear and specific issues ready for anyone to take them. Security labels May 17, 2015
@weaverryan weaverryan mentioned this issue May 17, 2015
Closed
@jeroenseegers jeroenseegers mentioned this issue Dec 1, 2015
Closed
xabbuh added a commit that referenced this issue Dec 9, 2015
@xabbuh
minor #5943 Add tip for when returning null from createToken() (j…
7186d17 
…eroenseegers)

This PR was submitted for the 2.8 branch but it was merged into the 2.7 branch instead (closes #5943).

Discussion
----------

Add tip for when returning `null` from `createToken()`

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | all
| Fixed tickets | #5264

Commits
-------

1337b31 Add caution when returning null from createToken()
@xabbuh xabbuh closed this as completed Dec 9, 2015
@xabbuh
Copy link
Member

xabbuh commented Dec 10, 2015

reopening here as we may want to put some more clarification here (see #5943 (comment))

@xabbuh xabbuh reopened this Dec 10, 2015
@javiereguiluz javiereguiluz mentioned this issue Aug 1, 2017
Closed
@javiereguiluz javiereguiluz added the hasPR A Pull Request has already been submitted for this issue. label Aug 1, 2017
xabbuh added a commit that referenced this issue Aug 29, 2017
@xabbuh
minor #8247 Improved a help note about authentication providers (javi…
396aee9 
…ereguiluz)

This PR was squashed before being merged into the 2.7 branch (closes #8247).

Discussion
----------

Improved a help note about authentication providers

This should fix #5264.

Commits
-------

bf26484 Improved a help note about authentication providers
@xabbuh xabbuh closed this as completed Aug 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
actionable Clear and specific issues ready for anyone to take them. good first issue Ideal for your first contribution! (some Symfony experience may be required) hasPR A Pull Request has already been submitted for this issue. Security
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@javiereguiluz @weaverryan @xabbuh