8000 [ErrorHandler] Add debug argument to decide whether debug page is shown or not by yceruto · Pull Request #35464 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

[ErrorHandler] Add debug argument to decide whether debug page is shown or not #35464

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
Jan 27, 2020

Conversation

yceruto
Copy link
Member
@yceruto yceruto commented Jan 24, 2020
Q A
Branch? 4.4
Bug fix? yes
New feature? no
Deprecations? no
Tickets Fix #35448
License MIT
Doc PR -

This ensures that the debug page (with stack trace) won't be (by default) displayed in non-CLI context when an early error occurs (after FB::boot()) in non-debug mode (prod). And Debug::enable() will enable it explicitly.

@yceruto yceruto force-pushed the error_handler_debug branch 2 times, most recently from a8718a8 to bc045a0 Compare January 26, 2020 20:15
@yceruto yceruto force-pushed the error_handler_debug branch 2 times, most recently from c749df4 to cf80224 Compare January 26, 2020 20:34
@yceruto
Copy link
Member Author
yceruto commented Jan 26, 2020

@nicolas-grekas I've simplied it a bit more.
It should be ready now.

Copy link
Member
@nicolas-grekas nicolas-grekas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@nicolas-grekas
Copy link
Member

Thank you @yceruto.

nicolas-grekas added a commit that referenced this pull request Jan 27, 2020
…page is shown or not (yceruto)

This PR was merged into the 4.4 branch.

Discussion
----------

[ErrorHandler] Add debug argument to decide whether debug page is shown or not

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

This ensures that the debug page (with stack trace) won't be (by default) displayed in non-CLI context when an early error occurs (after FB::boot()) in non-debug mode (prod). And `Debug::enable()` will enable it explicitly.

Commits
-------

cf80224 Added debug argument to decide if debug page should be shown or not
@nicolas-grekas nicolas-grekas merged commit cf80224 into symfony:4.4 Jan 27, 2020
@yceruto yceruto deleted the error_handler_debug branch January 27, 2020 13:19
This was referenced Jan 31, 2020
PhilETaylor pushed a commit to PhilETaylor/symfony that referenced this pull request Sep 6, 2023
… debug page is shown or not (yceruto)

This PR was merged into the 4.4 branch.

Discussion
----------

[ErrorHandler] Add debug argument to decide whether debug page is shown or not

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

This ensures that the debug page (with stack trace) won't be (by default) displayed in non-CLI context when an early error occurs (after FB::boot()) in non-debug mode (prod). And `Debug::enable()` will enable it explicitly.

Commits
-------

cf80224 Added debug argument to decide if debug page should be shown or not
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.

4 participants
0