8000 [Process] Add explanation how to make the Process survive a response by tristanbes · Pull Request #7158 · symfony/symfony-docs · GitHub
[go: up one dir, main page]

Skip to content

[Process] Add explanation how to make the Process survive a response #7158

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 1,233 commits into from

Conversation

tristanbes
Copy link
Contributor

Fixes #7151

It confuses user that could be tempted to use the Asynchronous Process feature to achieve long running task after the Response is sent.

The result is the following :
from no code execution (if you send a Response really fast)
to partial code execution

weaverryan and others added 30 commits August 1, 2016 21:31
* 2.7: (51 commits)
  this sounds just a little bit more natural
  Update dynamic_form_modification.rst
  Remove colloquialism "hold on"
  fixing reference to deprecated class
  removing use statement for consistency with other places in this doc
  fix FlattenException namespace
  Tweaks thanks to close review from Javier and @snoek09!
  Remove AcmeDemoBundle references
  Fix reference to output object
  Subject-verb agreement
  Fix tense and sentence length
  Fix typo in external_parameters.rst
  line break
  Removing the alias stuff - not required after symfony/symfony#17074
  Fixed the main index page redirections
  Fixed the redirection for "upgrade" articles
  Added all the missing "index pages" redirections
  Fixed a Console article redirection
  Fixed the redirection of the previous "performance" book chapter
  Fixed a DI redirection
  ...
* 2.8: (52 commits)
  this sounds just a little bit more natural
  Update dynamic_form_modification.rst
  Remove colloquialism "hold on"
  fixing reference to deprecated class
  removing use statement for consistency with other places in this doc
  fix FlattenException namespace
  Tweaks thanks to close review from Javier and @snoek09!
  Remove AcmeDemoBundle references
  Fix reference to output object
  Subject-verb agreement
  fixed missing level in namespace
  Fix tense and sentence length
  Fix typo in external_parameters.rst
  line break
  Removing the alias stuff - not required after symfony/symfony#17074
  Fixed the main index page redirections
  Fixed the redirection for "upgrade" articles
  Added all the missing "index pages" redirections
  Fixed a Console article redirection
  Fixed the redirection of the previous "performance" book chapter
  ...
* 3.0: (52 commits)
  this sounds just a little bit more natural
  Update dynamic_form_modification.rst
  Remove colloquialism "hold on"
  fixing reference to deprecated class
  removing use statement for consistency with other places in this doc
  fix FlattenException namespace
  Tweaks thanks to close review from Javier and @snoek09!
  Remove AcmeDemoBundle references
  Fix reference to output object
  Subject-verb agreement
  fixed missing level in namespace
  Fix tense and sentence length
  Fix typo in external_parameters.rst
  line break
  Removing the alias stuff - not required after symfony/symfony#17074
  Fixed the main index page redirections
  Fixed the redirection for "upgrade" articles
  Added all the missing "index pages" redirections
  Fixed a Console article redirection
  Fixed the redirection of the previous "performance" book chapter
  ...
* 3.1: (52 commits)
  this sounds just a little bit more natural
  Update dynamic_form_modification.rst
  Remove colloquialism "hold on"
  fixing reference to deprecated class
  removing use statement for consistency with other places in this doc
  fix FlattenException namespace
  Tweaks thanks to close review from Javier and @snoek09!
  Remove AcmeDemoBundle references
  Fix reference to output object
  Subject-verb agreement
  fixed missing level in namespace
  Fix tense and sentence length
  Fix typo in external_parameters.rst
  line break
  Removing the alias stuff - not required after symfony/symfony#17074
  Fixed the main index page redirections
  Fixed the redirection for "upgrade" articles
  Added all the missing "index pages" redirections
  Fixed a Console article redirection
  Fixed the redirection of the previous "performance" book chapter
  ...
Sinced Sy3 they are under `var` instead of `app`.
…ana-gligor)

This PR was submitted for the 3.1 branch but it was merged into the 3.0 branch instead (closes symfony#6825).

Discussion
----------

update `cache` and `logs` folder locations

Since Sy3 they are under `var` instead of `app`.

Commits
-------

396cf06 update `cache` and `logs` folder locations
…lchimik)

This PR was merged into the 3.1 branch.

Discussion
----------

Simplex\Framework - add argumentResolver property

Commits
-------

2611798 Simplex\Framework - add argumentResolver property
…r (alchimik)

This PR was merged into the 3.1 branch.

Discussion
----------

Simplex\Framework contructor - remove extra parameter

Commits
-------

c5a4ea3 Simplex\Framework contructor - extra parameter
…re in 2.8

In the example code of the custom error page, there was a comment reference to the `Avoiding Exceptions when Using Security Functions in Error Templates' section, that lived in the pre-2.8 documentation. That subsection is no longer there.
This PR was submitted for the master branch but it was merged into the 2.8 branch instead (closes symfony#6850).

Discussion
----------

Remove outdated reference

Finishes symfony#6771

Original description:

 > In the example code of the custom error page, there was a comment reference to the `Avoiding Exceptions when Using Security Functions in Error Templates' section, that lived in the pre-2.8 documentation. That subsection is no longer there.

Commits
-------

21fcb72 custom error pages - Reference to security functions warning is no more in 2.8
The first argument for SQLite3Cache should be of type SQLite3 instead of a string
…wimme002)

This PR was merged into the 3.1 branch.

Discussion
----------

Fix Cache Pools: SQLite3Cache constructor argument

The first argument for SQLite3Cache should be of type SQLite3 instead of a string

https://github.com/doctrine/cache/blob/master/lib/Doctrine/Common/Cache/SQLite3Cache.php#L68

Commits
-------

1a8c9ab Fix Cache Pools: SQLite3Cache constructor argument
This PR was merged into the 3.1 branch.

Discussion
----------

Update argument_value_resolver.rst

Commits
-------

1707db1 Update argument_value_resolver.rst
Conflicts:
	components/form.rst
xabbuh and others added 26 commits November 5, 2016 11:50
This PR was merged into the master branch.

Discussion
----------

[symfony#6947] some minor tweaks

Commits
-------

f865f93 [symfony#6947] some minor tweaks
* 2.7:
  [symfony#6961] add additional config formats
  [symfony#7104] minor typo fix
  Fixed minor syntax issues and typos
  more precision on class name, and lazy service
  add warning about the limitation on using swiftmail with file spool and lazy loading
  Removed the proposed note and updated the title
  Added note on ODM id notation being different
  [symfony#7099] remove trailing whitespaces
  Clarify Process::wait() callback behaviour
  Minor text fix - wrong submit button label (Forms)
  [Doctrine] Slave/Master configuration options
  Fix broken link
  Fix typo
  Add missing parenthesis for methods and a few minor tweaks
  [Doctrine] Exception note about functions with named managers
* 2.8:
  [symfony#6961] add additional config formats
  [symfony#7104] minor typo fix
  Fixed minor syntax issues and typos
  more precision on class name, and lazy service
  add warning about the limitation on using swiftmail with file spool and lazy loading
  Removed the proposed note and updated the title
  Added note on ODM id notation being different
  [symfony#7099] remove trailing whitespaces
  Clarify Process::wait() callback behaviour
  Minor text fix - wrong submit button label (Forms)
  [Doctrine] Slave/Master configuration options
  Fix broken link
  Fix typo
  Add missing parenthesis for methods and a few minor tweaks
  [Doctrine] Exception note about functions with named managers
* 3.1:
  [symfony#6961] add additional config formats
  [symfony#7104] minor typo fix
  Fixed minor syntax issues and typos
  more precision on class name, and lazy service
  add warning about the limitation on using swiftmail with file spool and lazy loading
  Removed the proposed note and updated the title
  Added note on ODM id notation being different
  [symfony#7099] remove trailing whitespaces
  Clarify Process::wait() callback behaviour
  Remove AssetsHelper from the templating component
  Minor text fix - wrong submit button label (Forms)
  [Doctrine] Slave/Master configuration options
  Fix broken link
  Fix typo
  Add missing parenthesis for methods and a few minor tweaks
  Update input.rst
  [Profiler] Fix rst typo
  [Doctrine] Exception note about functions with named managers
[ExpressionLanguage] Making cache PSR6 compliant
This PR was merged into the master branch.

Discussion
----------

Add dateinterval type reference

| Q | A |
| --- | --- |
| Doc fix? | no |
| New docs? | yes |
| Applies to | 3.2 |
| Fixed tickets | symfony#4816 |

Commits
-------

3d55585 Add dateinterval type reference
…n (jakzal)

This PR was merged into the master branch.

Discussion
----------

[DomCrawler] Document XPath expression evaluation

Documents symfony/symfony#19430

Commits
-------

156b047 [DomCrawler] Document XPath expression evaluation
…option (ogizanagi)

This PR was merged into the master branch.

Discussion
----------

[DoctrineBridge] Document the new `entityClass` option

Waiting for symfony/symfony#15002 to be merged.

Commits
-------

a9c2fc2 [DoctrineBridge] Document the new `entityClass` option
…imeNormalizer::denormalize (teohhanhui)

This PR was merged into the master branch.

Discussion
----------

[Serializer] Support specifying format for DateTimeNormalizer::denormalize

Waiting for symfony/symfony#20217 to be merged.

Commits
-------

57bf969 [Serializer] Support specifying format for DateTimeNormalizer::denormalize
This PR was merged into the master branch.

Discussion
----------

Update caching.rst

Updating doc for this PR Making cache PSR6 compliant : [symfony#19741](symfony/symfony#19741)

Commits
-------

f03f2f3 Update caching.rst
Conflicts:
	form/direct_submit.rst
	testing/doctrine.rst
…gs (iltar)

This PR was merged into the master branch.

Discussion
----------

Added docs mentioning UserInterface in action args

Added notes about the `UserInterface` added in symfony/symfony#18510, was waiting for the in-deprecation of the `getUser()` method: symfony/symfony#19452.

Commits
-------

7849fa2 Added docs mentioning UserInterface in action args
…esponse

Fixes symfony#7151

It confuses user that could be tempted to use the Asynchronous Process feature to achieve long running task after the Response is sent.

The result is the following :
from no code execution (if you send a Response really fast)
to partial code execution
@tristanbes tristanbes changed the base branch from master to 2.7 November 22, 2016 11:50
@tristanbes tristanbes closed this Nov 22, 2016
@tristanbes tristanbes deleted the patch-5 branch November 22, 2016 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0