JSON authentication listener docs #7081
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
1 task
|
@javiereguiluz great improvement, thanks! |
ogizanagi
reviewed
Oct 25, 2016
|
|
||
| .. tip:: | ||
|
|
||
| If you are storing users in some sort of a database, then you should consider |
There was a problem hiding this comment.
Is it really relevant here ? IMHO it does not have specifically much to do with the current article :/
| the following JSON document as body, the security system will automatically | ||
| check the user's credentials and either authenticate the user or throw an error:: | ||
|
|
||
| .. code-block:: json |
There was a problem hiding this comment.
For some reason, this block does not render well: http://pr-7081-acxnhhi-6qmocelev2lwe.eu.platform.sh/security/json_login_setup.html
(probably because of the double colon L.149)
fabpot
added a commit
to symfony/symfony
that referenced
this pull request
Dec 3, 2016
This PR was squashed before being merged into the 3.3-dev branch (closes #18952). Discussion ---------- [Security] Add a JSON authentication listener | Q | A | | --- | --- | | Branch? | master | | Bug fix? | no | | New feature? | yes | | BC breaks? | no | | Deprecations? | no | | Tests pass? | yes | | Fixed tickets | n/a | | License | MIT | | Doc PR | symfony/symfony-docs#7081 | Add a new authentication listener allowing to login by sending a JSON document like: `{"_username": "dunglas", "_password": "foo"}`. It is similar to the traditional form login (but take a JSON document as entry) and is convenient for APIs, especially used in combination with JWT. See api-platform/core#563 and lexik/LexikJWTAuthenticationBundle#123 (comment) for previous discussions. - [x] Add functional tests in security bundle Commits ------- 02178bc [Security] Add a JSON authentication listener
javiereguiluz
approved these changes
Dec 5, 2016
There was a problem hiding this comment.
👍
@dunglas thanks for implementing this nice feature and thanks for providing the docs.
|
Thank you very much for all these improvements @javiereguiluz |
xabbuh
added a commit
that referenced
this pull request
Dec 14, 2016
This PR was merged into the master branch. Discussion ---------- JSON authentication listener docs Docs for symfony/symfony#18952. Commits ------- b192ab3 Fixed a syntax issue 16ae3f6 Reworded and simplified the article 820f28e Fixed the name of the "username" property 6d018f6 Fixed a syntax error ab5259b Removed a tip which is not too relevant for the article d2dd895 Fixed the JSON format (this time for real) 6ff24da Fixed the JSON format 0961128 Show the simple example first and then explain the complex use case 68bd9a5 JSON authentication listener docs
xabbuh
added a commit
that referenced
this pull request
Dec 14, 2016
|
Thank you Kevin. I have merged your PR and made some minor tweaks in af1fb6b. Let me know if you disagree with any of them. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Docs for symfony/symfony#18952.