Skip to content

[Cache] fix memory leak when using PhpFilesAdapter #34896

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

Merged
merged 1 commit into from
Dec 10, 2019

Conversation

nicolas-grekas
Copy link
Member

Q A
Branch? 4.3
Bug fix? yes
New feature? no
Deprecations? no
Tickets Fix #34687
License MIT
Doc PR -

Similar to #34839 but for PhpFilesAdapter, as the "appendOnly" mode is a v4-only feature.

@nicolas-grekas nicolas-grekas added this to the 4.3 milestone Dec 9, 2019
@nicolas-grekas nicolas-grekas force-pushed the cache-phpfiles-leak branch 7 times, most recently from 98ea7b1 to c0af43c Compare December 9, 2019 19:47
nicolas-grekas added a commit that referenced this pull request Dec 10, 2019
…s-grekas)

This PR was merged into the 4.3 branch.

Discussion
----------

[Cache] fix memory leak when using PhpFilesAdapter

| Q             | A
| ------------- | ---
| Branch?       | 4.3
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | Fix #34687
| License       | MIT
| Doc PR        | -

Similar to #34839 but for `PhpFilesAdapter`, as the "appendOnly" mode is a v4-only feature.

Commits
-------

0b46226 [Cache] fix memory leak when using PhpFilesAdapter
@nicolas-grekas nicolas-grekas merged commit 0b46226 into symfony:4.3 Dec 10, 2019
@nicolas-grekas nicolas-grekas deleted the cache-phpfiles-leak branch December 13, 2019 13:57
This was referenced Dec 19, 2019
@bendavies
Copy link
Contributor

bendavies commented Dec 29, 2019

Any chance of a 4.3 release with this included? A 4.4 and 5.0 release was made with this, but not 4.3.

@fabpot fabpot mentioned this pull request Jan 21, 2020
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.

3 participants