Skip to content

[ErrorRenderer] Allow disabling debug content in debug mode (preview mode) #32693

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
Jul 24, 2019

Conversation

yceruto
Copy link
Member

@yceruto yceruto commented Jul 23, 2019

Q A
Branch? 4.4
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT

Required by #31398 to show a preview mode of the error for each content format.

Usage: https://github.com/symfony/symfony/pull/31398/files#diff-9ff5216ab011f5e48c7835d4138bf825R42

Note that you can't enable the debug content in non-debug mode via X-Debug. I also added more tests.

@yceruto yceruto force-pushed the error_renderer_x_debug branch from 84949d4 to c19d8bf Compare July 24, 2019 02:48
@yceruto yceruto force-pushed the error_renderer_x_debug branch from c19d8bf to a6bef5e Compare July 24, 2019 03:05
@Tobion
Copy link
Contributor

Tobion commented Jul 24, 2019

Thank you @yceruto.

@Tobion Tobion merged commit a6bef5e into symfony:4.4 Jul 24, 2019
Tobion added a commit that referenced this pull request Jul 24, 2019
…ode (preview mode) (yceruto)

This PR was merged into the 4.4 branch.

Discussion
----------

[ErrorRenderer] Allow disabling debug content in debug mode (preview mode)

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT

Required by #31398 to show a preview mode of the error for each content format.

Usage: https://github.com/symfony/symfony/pull/31398/files#diff-9ff5216ab011f5e48c7835d4138bf825R42

Note that you can't enable the debug content in non-debug mode via `X-Debug`. I also added more tests.

Commits
-------

a6bef5e Allow disabling debug content in debug mode (preview mode)
@yceruto yceruto deleted the error_renderer_x_debug branch July 24, 2019 12:00
@nicolas-grekas nicolas-grekas modified the milestones: next, 4.4 Oct 27, 2019
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