-
Notifications
You must be signed in to change notification settings - Fork 18
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
Refactor the whole application kernel #396
Conversation
3239e86
to
8952897
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice refacto 💛
7124437
to
2b6c0ba
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, the PR is now ready for review
@@ -78,6 +79,7 @@ private static function configureDebug(): ErrorHandler | |||
|
|||
AbstractCloner::$defaultCasters[self::class] = ['Symfony\Component\VarDumper\Caster\StubCaster', 'cutInternals']; | |||
AbstractCloner::$defaultCasters[AfterApplicationInitializationEvent::class] = ['Symfony\Component\VarDumper\Caster\StubCaster', 'cutInternals']; | |||
AbstractCloner::$defaultCasters[DescriptorsCollection::class] = ['Symfony\Component\VarDumper\Caster\StubCaster', 'cutInternals']; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Much better display when running castor -vvv
Is there any way to forbid it? I think it could lead to security issues (someone publish a castor random task that sounds useful, but this task import another one (repeat N times) and the last one contain malicious code. |
Indeed, but it's already the case with a "simple" import. I mean, this PR does not allow more security issue than the one already present |
import
from a git repository #395