Skip to content

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