Skip to content

Container optimizations #52191

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

Draft
wants to merge 17 commits into
base: master
Choose a base branch
from
Draft

Container optimizations #52191

wants to merge 17 commits into from

Conversation

icewind1991
Copy link
Member

Another shot at #36613 since the server container is still a non-insignificant cost of requests

@icewind1991 icewind1991 added the 2. developing Work in progress label Apr 14, 2025
if ($item instanceof ServiceFactory) {
return $item->get();
} elseif (is_callable($item)) {
$this->items[$name] = $item($this);

Check failure

Code scanning / Psalm

TaintedCallable Error

Detected tainted text
if ($item instanceof ServiceFactory) {
return $item->get();
} elseif (is_callable($item)) {
$this->items[$name] = $item($this);

Check failure

Code scanning / Psalm

TaintedCallable Error

Detected tainted text
@icewind1991 icewind1991 force-pushed the container-optimizations branch from 6774d51 to c0c57a9 Compare April 15, 2025 18:55
…r when instances are already created

Signed-off-by: Robin Appelman <[email protected]>
Signed-off-by: Robin Appelman <[email protected]>
Signed-off-by: Robin Appelman <[email protected]>
Signed-off-by: Robin Appelman <[email protected]>
@icewind1991 icewind1991 force-pushed the container-optimizations branch from c0c57a9 to df99687 Compare April 16, 2025 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2. developing Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant