Skip to content

[Logging] Fix incorrect PHP example to exclude HTTP code with Monolog #15449

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
Jun 18, 2021

Conversation

ker0x
Copy link
Contributor

@ker0x ker0x commented Jun 18, 2021

With the previous example, when running composer run-script --no-dev post-install-cmd we get the following error:

Executing script cache:clear [KO]
 [KO]
Script cache:clear returned with error code 1
!!  
!!  In FileLoader.php line 174:
!!                                                                                 
!!    The following keys are not supported by "Symfony\Config\Monolog\HandlerConf  
!!    ig\ExcludedHttpCodeConfig": 0, 1 in /home/ker0x/dev/imdb/config/packages/pr  
!!    od/monolog.php (which is being imported from "/home/ker0x/dev/imdb/src/Kern  
!!    el.php").                                                                    
!!                                                                                 
!!  
!!  In ExcludedHttpCodeConfig.php line 57:
!!                                                                                 
!!    The following keys are not supported by "Symfony\Config\Monolog\HandlerConf  
!!    ig\ExcludedHttpCodeConfig": 0, 1                                             
!!                                                                                 
!!  
!!  
Script @auto-scripts was called via post-install-cmd

@javiereguiluz
Copy link
Member

Good catch, thanks Romain.

@javiereguiluz javiereguiluz merged commit 12d52ea into symfony:5.3 Jun 18, 2021
@ker0x ker0x deleted the bugfix/monolog-excluded-http-code branch June 22, 2021 09:18
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.

3 participants