Thanks to visit codestin.com
Credit goes to github.com

Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: symfony/http-kernel
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v4.4.47
Choose a base ref
...
head repository: symfony/http-kernel
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v4.4.48
Choose a head ref
  • 6 commits
  • 5 files changed
  • 4 contributors

Commits on Oct 12, 2022

  1. Bump Symfony version to 4.4.48

    fabpot committed Oct 12, 2022
    Configuration menu
    Copy the full SHA
    d84ce93 View commit details
    Browse the repository at this point in the history

Commits on Oct 18, 2022

  1. Configuration menu
    Copy the full SHA
    abc1357 View commit details
    Browse the repository at this point in the history
  2. bug #47878 [HttpKernel] Remove EOL when using error_log() in HttpKern…

    …el Logger (cyve)
    
    This PR was squashed before being merged into the 4.4 branch.
    
    Discussion
    ----------
    
    [HttpKernel] Remove EOL when using error_log() in HttpKernel Logger
    
    | Q             | A
    | ------------- | ---
    | Branch?       | 4.4
    | Bug fix?      | yes
    | New feature?  | no
    | Deprecations? | maybe
    | Tickets       |
    | License       | MIT
    | Doc PR        |
    
    Hello there !
    
    Since PR symfony/symfony#36855, when using `Symfony\Component\HttpKernel\Log\Logger`  with argument `$output = null`. The logger use the `error_log()` method to log messages. But in the log file, messages are separated by an empty line because the default formatter (`Logger::format()`) adds an extra EOL char at the end of the message.
    
    This fix prevents the logger to add an extra EOL char when it uses `error_log()`.
    
    Possible BC : if someone use the logger with a custom formatter that already add a EOL char at the end of the message, there will be a empty line between messages in the log file. In my opinion, this is minor enough not to worry about it ;-)
    
    Thanks for your review :-)
    
    Commits
    -------
    
    69cf83ea1a [HttpKernel] Remove EOL when using error_log() in HttpKernel Logger
    nicolas-grekas committed Oct 18, 2022
    Configuration menu
    Copy the full SHA
    9a34f1a View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    3f61170 View commit details
    Browse the repository at this point in the history
  4. bug #47857 [HttpKernel] Fix empty request stack when terminating with…

    … exception (krzyc)
    
    This PR was squashed before being merged into the 4.4 branch.
    
    Discussion
    ----------
    
    [HttpKernel] Fix empty request stack when terminating with exception
    
    | Q             | A
    | ------------- | ---
    | Branch?       | 4.4, 5.4, 6.0, 6.1, 6.2
    | Bug fix?      | yes
    | New feature?  | no
    | Deprecations? | no
    | Tickets       | Fix #47577
    | License       | MIT
    | Doc PR        |
    
    After symfony/symfony#47358 the RequestStack is empty when request terminates with exception, which prevents SecurityDataCollector to generate logout URL and generates fatal error.
    
    Commits
    -------
    
    e4d6e7b4ba [HttpKernel] Fix empty request stack when terminating with exception
    nicolas-grekas committed Oct 18, 2022
    Configuration menu
    Copy the full SHA
    26989b2 View commit details
    Browse the repository at this point in the history

Commits on Oct 28, 2022

  1. Update VERSION for 4.4.48

    fabpot committed Oct 28, 2022
    Configuration menu
    Copy the full SHA
    a6d5229 View commit details
    Browse the repository at this point in the history
Loading