Split Security into Authentication & Authorization #5367
Merged
Conversation
This file contains 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
|
you need to update the map file so that pages appear in the same order in the map than here. Otherwise the prev and next links will be wrong |
|
@stof which one do you mean? I've updated 1 map file. |
| @@ -1,6 +1,8 @@ | |||
| Security | |||
| ======== | |||
|
|
|||
| * Authentication | |||
There was a problem hiding this comment.
I would love to use subsections:
Authentication
--------------
...There was a problem hiding this comment.
That might indeed be a nice thing, I'll update this.
| multiple_user_providers | ||
|
|
||
|
|
There was a problem hiding this comment.
we don't need two blank lines here
linaori
force-pushed
the
feature/security-split
branch
from
1d04f6d to
825f447
Compare
|
👍 |
| @@ -143,27 +143,30 @@ | |||
| * :doc:`/cookbook/routing/redirect_trailing_slash` | |||
| * :doc:`/cookbook/routing/extra_information` | |||
|
|
|||
| * :doc:`/cookbook/security/index` | |||
| * :doc:`Authentication </cookbook/security/index>` | |||
There was a problem hiding this comment.
How about: Security Authentication
and below Security Authorization
so that when people search for "security", then find these
There was a problem hiding this comment.
Per our conversation on IRC:
Security Authentication (Identifying/Logging in the User)
Security Authorization (Denying Access)
weaverryan
added a commit
that referenced
this pull request
Jul 7, 2015
This PR was merged into the 2.3 branch. Discussion ---------- Split Security into Authentication & Authorization | Q | A | ------------- | --- | Doc fix? | yes | New docs? | no | Applies to | all | Fixed tickets | ~ I have divided the security part into Authentication and Authorization. This is done visually only, structure wise they are still together. There's still an issue regarding the sorting, it will be a lot clearer if the lists are re-sorted but I will do this in another PR once this is done. I think I've update all the index pages that mention this, but if I missed something, I can add that. Commits ------- 825f447 Changed Authorization and Authentication to use subsections 2e2068a Split Security into Authentication & Authorization
weaverryan
added a commit
that referenced
this pull request
Jul 7, 2015
weaverryan
added a commit
that referenced
this pull request
Jul 7, 2015
* 2.3: Reordered the configuration blocks of the first example Fixed some errors and made some simplifications Added the "payload" option back (was removed by mistake) Fixed the order of the examples Simplified the first example and added more examples [Validator] Updated documentation of URL validator Fixed typos Removed duplication and moved a caution message [Console] Added a cookbook entry on invoking other commands [#5367] Making the titles more searchable fix #5487 Changed Authorization and Authentication to use subsections Split Security into Authentication & Authorization
weaverryan
added a commit
that referenced
this pull request
Jul 7, 2015
* 2.6: Reordered the configuration blocks of the first example Fixed some errors and made some simplifications Added the "payload" option back (was removed by mistake) Fixed the order of the examples Simplified the first example and added more examples [Validator] Updated documentation of URL validator Fixed typos Removed duplication and moved a caution message [Console] Added a cookbook entry on invoking other commands [#5367] Making the titles more searchable fix #5487 Changed Authorization and Authentication to use subsections Split Security into Authentication & Authorization
weaverryan
added a commit
that referenced
this pull request
Jul 7, 2015
* 2.7: [#5426] Porting the checkDNS option to 2.7 only Reordered the configuration blocks of the first example Fixed some errors and made some simplifications Added the "payload" option back (was removed by mistake) Fixed the order of the examples Simplified the first example and added more examples [Validator] Updated documentation of URL validator Fixed typos Removed duplication and moved a caution message [Console] Added a cookbook entry on invoking other commands [#5367] Making the titles more searchable fix #5487 [FrameworkBundle] Update serializer configuration reference Changed Authorization and Authentication to use subsections Split Security into Authentication & Authorization
weaverryan
added a commit
that referenced
this pull request
Jul 7, 2015
* 2.7: [#5426] Porting the checkDNS option to 2.7 only Reordered the configuration blocks of the first example Fixed some errors and made some simplifications Added the "payload" option back (was removed by mistake) Fixed the order of the examples Simplified the first example and added more examples [Validator] Updated documentation of URL validator Fixed typos Removed duplication and moved a caution message [Console] Added a cookbook entry on invoking other commands [#5367] Making the titles more searchable fix #5487 [FrameworkBundle] Update serializer configuration reference Changed Authorization and Authentication to use subsections Split Security into Authentication & Authorization
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.
I have divided the security part into Authentication and Authorization. This is done visually only, structure wise they are still together. There's still an issue regarding the sorting, it will be a lot clearer if the lists are re-sorted but I will do this in another PR once this is done.
I think I've update all the index pages that mention this, but if I missed something, I can add that.