Skip to content

[WCM] Idle sessions expiration cookbook recipe. #4576

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 4 commits into from

Conversation

ajgarlag
Copy link
Contributor

@ajgarlag ajgarlag commented Dec 2, 2014

Q A
Doc fix? no
New docs? yes (PR symfony/symfony#12807)
Applies to 2.7
Fixed tickets

@xabbuh
Copy link
Member

xabbuh commented Dec 2, 2014

@ajgarlag Thank you for the pull request. FYI, I have added [WCM] to the title which means "Waiting Code Merge".

@xabbuh xabbuh changed the title Idle sessions expiration cookbook recipe. [WCM] Idle sessions expiration cookbook recipe. Dec 2, 2014
.. index::
single: Security; Expiration of idle sessions

Expiration of idle sessions
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be "Expiration of Idle Sessions" according to our headline capitalization rules (capitalize all words expect of close class words).

),
));


Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You have a double empty line here. Could you please remove one of them?

@wouterj
Copy link
Member

wouterj commented Dec 19, 2014

Thank you for submitting such detailed docs while doing a PR on the code, @ajgarlag! I've left some comments.

@ajgarlag
Copy link
Contributor Author

Thank you for reviewing it. I hope this can help to merge the PR soon.

.. index::
single: Security; Expiration of Idle sessions

Expiration of Idle sessions
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sessions (uppercased s)

@xabbuh xabbuh added this to the 3.2 milestone May 21, 2016
@wouterj wouterj removed this from the 3.2 milestone Nov 8, 2016
@xabbuh xabbuh added this to the 3.3 milestone Dec 30, 2016
@weaverryan weaverryan added the Waiting Code Merge Docs for features pending to be merged label Oct 6, 2017
@xabbuh xabbuh modified the milestones: 3.3, 4.1 Jan 29, 2018

.. code-block:: yaml

# app/config/security.yml
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

# config/packages/security.yaml instead of # app/config/security.yml


.. code-block:: xml

<!-- app/config/security.xml -->
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

# config/packages/security.xml


.. code-block:: php

// app/config/security.php
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

# config/packages/security.php

http://symfony.com/schema/dic/services/services-1.0.xsd">

<config>
<firewall>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you forgot here the firewall name


.. code-block:: yaml

# app/config/security.yml
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

# config/packages/security.yaml here

@xabbuh xabbuh removed this from the 4.1 milestone May 8, 2018
@xabbuh xabbuh added this to the next milestone May 8, 2018
@HeahDude HeahDude removed the On hold label Feb 7, 2020
@javiereguiluz
Copy link
Member

I'm closing this one because the related code PR was closed recently. Sorry!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Work Waiting Code Merge Docs for features pending to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants