[12.x] Add ability to flush state on Vite helper #55228
Merged
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.
There is a memory leak in Octane as the Vite class is a singleton which collects state on each request.
We cannot make the class a scoped instance, as it needs to store some registered information between requests.
This PR adds a
flush
method that we will then call in Octane to clear the state between executions.Octane PR: https://github.com/laravel/octane/pull/1016/files#diff-713c2b10a1e78ad1a57b948bbecdeace4afcef924eef6ce3c0899d2fa2fe7134L51-R52