Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[Debug][HttpKernel] remove frames added by DebugClassLoader in stack traces#29190
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
nicolas-grekas commentedNov 12, 2018
| Q | A |
|---|---|
| Branch? | master |
| Bug fix? | no |
| New feature? | yes |
| BC breaks? | no |
| Deprecations? | no |
| Tests pass? | yes |
| Fixed tickets | #28818 |
| License | MIT |
| Doc PR | - |
Uh oh!
There was an error while loading.Please reload this page.
2e7eaf6 to76e7944Compare
j4nr6n left a comment
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.
Tested and working in my scratch project. Looks really good. 👍
Tobion commentedNov 15, 2018
@j4nr6n the example you used is not affected by this change. This is about the deprecations coming from the DebugClassLoader which is not triggered in your example. |
fabpot commentedNov 16, 2018
Thank you@nicolas-grekas. |
…r in stack traces (nicolas-grekas)This PR was merged into the 4.2-dev branch.Discussion----------[Debug][HttpKernel] remove frames added by DebugClassLoader in stack traces| Q | A| ------------- | ---| Branch? | master| Bug fix? | no| New feature? | yes| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets |#28818| License | MIT| Doc PR | -Commits-------76e7944 [Debug][HttpKernel] remove frames added by DebugClassLoader in stack traces