Name already in use

…t (fabpot)

This PR was merged into the 6.3 branch.

Discussion
----------

[Validator] Make exception for all constraints consistent

| Q             | A
| ------------- | ---
| Branch?       | 6.3
| Bug fix?      | no
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tickets       | n/a
| License       | MIT
| Doc PR        | n/a

Let's be sure we're using Validator exception classes everywhere.

Commits
-------

cab49ef [Validator] Make exception for all constraints consistent

…IONS from superglobal arrays too (xabbuh)

This PR was merged into the 6.3 branch.

Discussion
----------

[ErrorHandler] try to read SYMFONY_PATCH_TYPE_DECLARATIONS from superglobal arrays too

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

Commits
-------

ef7e6b1 try to read SYMFONY_PATCH_TYPE_DECLARATIONS from superglobal arrays too

…orary directory (xabbuh)

This PR was merged into the 6.3 branch.

Discussion
----------

[HttpKernel] fix tests by using the actual system's temporary directory

| Q             | A
| ------------- | ---
| Branch?       | 6.3
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       |
| License       | MIT
| Doc PR        |

Commits
-------

deaa81c fix tests by using the actual system's temporary directory

…anel filters (javiereguiluz)

This PR was squashed before being merged into the 6.3 branch.

Discussion
----------

[WebProfilerBundle] [WebProfilerPanel] Update the logger panel filters

| Q             | A
| ------------- | ---
| Branch?       | 6.3
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | #48762 (comment)
| License       | MIT
| Doc PR        | -

This only changes the HTML contents of the logger profiler panel to make them valid (and keeps them accessible). No feature or behavior was changed.

Commits
-------

a8523cb [WebProfilerBundle] [WebProfilerPanel] Update the logger panel filters

…the test container (nicolas-grekas)

This PR was merged into the 6.3 branch.

Discussion
----------

[FrameworkBundle] Allow setting private services with the test container

| Q             | A
| ------------- | ---
| Branch?       | 6.3
| Bug fix?      | no
| New feature?  | yes
| Deprecations? | no
| Tickets       | -
| License       | MIT
| Doc PR        | -

This PR allows mocking a service for a test case. A typical example could be for HttpClient:

```php
static::getContainer()->set('example.client', new MockHttpClient(...));

// do stuff that will use `example.client` and will get mock responses instead of using the network
```

Commits
-------

30cb03f [FrameworkBundle] Allow setting private services with the test container

* 6.2:
  fix merge

This PR was merged into the 6.3 branch.

Discussion
----------

Leverage arrow function syntax for closure

| Q             | A
| ------------- | ---
| Branch?       | 6.3
| Bug fix?      | no
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tickets       | Fix #47658 <!-- prefix each issue number with "Fix #", no need to create an issue if none exists, explain below instead -->
| License       | MIT
| Doc PR        | <!-- required for new features -->

Rationale in the RFC [here](https://wiki.php.net/rfc/arrow_functions_v2#introduction)

It's also notable that using arrow function syntax rather than the classic one has been enforced in the past by symfony core member: #48069 (comment)

So this PR would be consistent.

Commits
-------

f5802d3 Leverage arrow function syntax for closure

…rs (nicolas-grekas)

This PR was merged into the 6.3 branch.

Discussion
----------

[HttpKernel] check preload items returned by cache warmers

| Q             | A
| ------------- | ---
| Branch?       | 6.3
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | Fix #47136
| License       | MIT
| Doc PR        | -

Commits
-------

6d90590 [HttpKernel] check preload items returned by cache warmers

…kets (PhilETaylor)

This PR was squashed before being merged into the 6.3 branch.

Discussion
----------

[Messenger] Allow password in redis dsn when using sockets

| Q             | A
| ------------- | ---
| Branch?       | 6.3
| Bug fix?      | no
| New feature?  | yes
| Deprecations? | no
| Tickets       | Fix #47393 `@Arakmar`
| License       | MIT
| Doc PR        |

When a messenger async transport is configured with Redis (unix socket with a password), it gives DSN parsing errors.

Using a Bunch of copy and pasted code blocks from the Redis cache adapter, this PR fixes the bug that doesn't parse a DSN such as

`redis://password@/var/run/redis.sock`

Replaces bad rebased #47475

Commits
-------

9af8ccf [Messenger] Allow password in redis dsn when using sockets

* 6.2:
  [DependencyInjection] fixes validation of non-scalar attribute values
  [SecurityBundle] Fix using same handler for multiple authenticators
  [DependencyInjection] Fix dump order of inlined deps
  [VarExporter] Fix exporting enums
  [HttpClient] Let curl handle content-length headers
  [Intl] Fix tests

…bute values (ju1ius)

This PR was squashed before being merged into the 6.2 branch.

Discussion
----------

[DependencyInjection] fixes validation of non-scalar attribute values

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

As a follow-up to #47364, this PR updates the `CheckDefinitionValidityPass` to allow (possibly nested) arrays of scalars in service tags attribute values.
Please see #48956 for context.

Commits
-------

88b3e15 [DependencyInjection] fixes validation of non-scalar attribute values

* 6.1:
  [SecurityBundle] Fix using same handler for multiple authenticators
  [DependencyInjection] Fix dump order of inlined deps
  [VarExporter] Fix exporting enums
  [HttpClient] Let curl handle content-length headers
  [Intl] Fix tests

* 6.0:
  [SecurityBundle] Fix using same handler for multiple authenticators
  [DependencyInjection] Fix dump order of inlined deps
  [VarExporter] Fix exporting enums
  [HttpClient] Let curl handle content-length headers
  [Intl] Fix tests

* 5.4:
  [SecurityBundle] Fix using same handler for multiple authenticators
  [DependencyInjection] Fix dump order of inlined deps
  [VarExporter] Fix exporting enums
  [HttpClient] Let curl handle content-length headers

…nticators (RobertMe)

This PR was merged into the 5.4 branch.

Discussion
----------

[SecurityBundle] Fix using same handler for multiple authenticators

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

Using a reference for custom success/failure handler breaks using the same service for multiple authenticators. This as the options will be overriden by any later authenticators. So use a ChildDefinition instead so every authenticator gets a unique instance.

Commits
-------

c589905 [SecurityBundle] Fix using same handler for multiple authenticators

Using a reference for custom success/failure handler breaks using the
same service for multiple authenticators. This as the options will be
overriden by any later authenticators. So use a ChildDefinition instead
so every authenticator gets a unique instance.

Fixes: #48923

…las-grekas)

This PR was merged into the 5.4 branch.

Discussion
----------

[DependencyInjection] Fix dump order of inlined deps

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

Fix confirmed on the provided reproducers.
The updated fixture makes more sense also.

Commits
-------

1d01452 [DependencyInjection] Fix dump order of inlined deps

…as-grekas)

This PR was merged into the 5.4 branch.

Discussion
----------

[HttpClient] Let curl handle content-length headers

| Q             | A
| ------------- | ---
| Branch?       | 5.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | Fix #47973, #46499, #46823
| License       | MIT
| Doc PR        | -

I'm not sure why (maybe some older and buggy curl versions?) but it looks like curl might send content-length headers to proxies.

This didn't happen before #45261 so I'm trying to use curl in the same was as we did by the time, aka letting it handle content-length header when possible.

This is something that I already attempted in #45814 but was partly reverted in #45998. The attached patch is free from such issue.

Commits
-------

5b27dc2 [HttpClient] Let curl handle content-length headers

This PR was merged into the 5.4 branch.

Discussion
----------

[VarExporter] Fix exporting enums

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

Commits
-------

4574d87 [VarExporter] Fix exporting enums

…rawler::innerText()` and make it return the first non-empty text (otsch)

This PR was merged into the 6.3 branch.

Discussion
----------

[DomCrawler] Add argument `$normalizeWhitespace` to `Crawler::innerText()` and make it return the first non-empty text

This is a new PR instead of #48684 with target branch 6.3 as requested.

| Q             | A
| ------------- | ---
| Branch?       | 6.3
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | Fix #48682
| License       | MIT

Commits
-------

bb0c214 [DomCrawler] Add argument `$normalizeWhitespace` to `Crawler::innerText()` and make it return the first non-empty text