… in RedisTrait (wikando-ck)"

This reverts commit 6d95721, reversing
changes made to 101a5cc.

…isTrait" (nicolas-grekas)

This PR was merged into the 6.4 branch.

Discussion
----------

[Cache] Revert "Fix DSN auth not passed to clusters in RedisTrait"

| Q             | A
| ------------- | ---
| Branch?       | 6.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        | Fix #63261
| License       | MIT

This reverts #62852 (preserving the CS improvements) /cc `@ckrack` `@Vilx2`
See #63261 for why.

The `auth` query parameter should be used instead.

Commits
-------

b68546b Revert "bug #62852 [Cache] Fix DSN auth not passed to clusters in RedisTrait (wikando-ck)"

* 6.4:
  Revert "bug #62852 [Cache] Fix DSN auth not passed to clusters in RedisTrait (wikando-ck)"
  [Messenger][Cache] Fix forwarding SSL settings to the redis sentinel
  fix compatibility with PCRE < 10.40
  [Intl] Update data to ICU 78.2
  [Messenger] Fix AMQP heartbeat reconnection during in-flight message handling
  [Messenger] Optimize serialized size of ErrorDetailsStamp

* 7.4:
  [JsonStreamer] Fix DateTime handling in union types
  Revert "bug #62852 [Cache] Fix DSN auth not passed to clusters in RedisTrait (wikando-ck)"
  [Messenger][Cache] Fix forwarding SSL settings to the redis sentinel
  [FrameworkBundle] Fix autoconfiguring controllers using legacy Route annotations as attributes
  fix compatibility with PCRE < 10.40
  fix forward compatibility with JsonStreamer 8.1+
  [Intl] Update data to ICU 78.2
  [Messenger] Fix AMQP heartbeat reconnection during in-flight message handling
  [JsonStreamer] Fix composite node provider arguments in stream mode
  [PropertyInfo] Fix union with mixed handling
  [Messenger] Optimize serialized size of ErrorDetailsStamp
  fix compatibility with DBAL 4.4+

* 8.0:
  [JsonStreamer] Fix DateTime handling in union types
  Revert "bug #62852 [Cache] Fix DSN auth not passed to clusters in RedisTrait (wikando-ck)"
  [Messenger][Cache] Fix forwarding SSL settings to the redis sentinel
  [FrameworkBundle] Fix autoconfiguring controllers using legacy Route annotations as attributes
  fix compatibility with PCRE < 10.40
  fix forward compatibility with JsonStreamer 8.1+
  [Intl] Update data to ICU 78.2
  [Messenger] Fix AMQP heartbeat reconnection during in-flight message handling
  [JsonStreamer] Fix composite node provider arguments in stream mode
  [PropertyInfo] Fix union with mixed handling
  [Messenger] Optimize serialized size of ErrorDetailsStamp
  fix compatibility with DBAL 4.4+

…rait

Populate $params["auth"] from DSN-parsed credentials for classes that
receive auth via their constructor or connect call (Redis, RedisCluster,
Relay), while explicitly excluding Sentinel mode where auth is applied
post-connect to the resolved master node.

This re-applies the fix from symfony#62852 which was reverted in symfony#63306 due to
a regression for Redis Sentinel users (symfony#63261). The original fix
unconditionally set $params["auth"] from DSN credentials, which caused
them to be incorrectly passed to Sentinel servers that do not require
authentication.

The new shouldPopulateAuthFromDsn() method centralizes this decision by
checking both the client class and the absence of Sentinel configuration.

As documented at https://symfony.com/doc/current/components/cache/adapters/redis_adapter.html#configure-the-connection,
the DSN supports password authentication (e.g. redis://user:password@host),
which should work without requiring the undocumented auth query parameter.

…rait

Populate $params["auth"] from DSN-parsed credentials for classes that
receive auth via their constructor or connect call (Redis, RedisCluster,
Relay), while explicitly excluding Sentinel mode where auth is applied
post-connect to the resolved master node.

This re-applies the fix from symfony#62852 which was reverted in symfony#63306 due to
a regression for Redis Sentinel users (symfony#63261). The original fix
unconditionally set $params["auth"] from DSN credentials, which caused
them to be incorrectly passed to Sentinel servers that do not require
authentication.

The new shouldPopulateAuthFromDsn() method centralizes this decision by
checking both the client class and the absence of Sentinel configuration.

As documented at https://symfony.com/doc/current/components/cache/adapters/redis_adapter.html#configure-the-connection,
the DSN supports password authentication (e.g. redis://user:password@host),
which should work without requiring the undocumented auth query parameter.

…rait

Populate $params["auth"] from DSN-parsed credentials for classes that
receive auth via their constructor or connect call (Redis, RedisCluster,
Relay), while explicitly excluding Sentinel mode where auth is applied
post-connect to the resolved master node.

This re-applies the fix from symfony#62852 which was reverted in symfony#63306 due to
a regression for Redis Sentinel users (symfony#63261). The original fix
unconditionally set $params["auth"] from DSN credentials, which caused
them to be incorrectly passed to Sentinel servers that do not require
authentication.

As documented at https://symfony.com/doc/current/components/cache/adapters/redis_adapter.html#configure-the-connection,
the DSN supports password authentication (e.g. redis://user:password@host),
which should work without requiring the undocumented auth query parameter.

…rait

Populate $params["auth"] from DSN-parsed credentials for classes that
receive auth via their constructor or connect call (Redis, RedisCluster,
Relay), while explicitly excluding Sentinel mode where auth is applied
post-connect to the resolved master node.

This re-applies the fix from symfony#62852 which was reverted in symfony#63306 due to
a regression for Redis Sentinel users (symfony#63261). The original fix
unconditionally set $params["auth"] from DSN credentials, which caused
them to be incorrectly passed to Sentinel servers that do not require
authentication.

As documented at https://symfony.com/doc/current/components/cache/adapters/redis_adapter.html#configure-the-connection,
the DSN supports password authentication (e.g. redis://user:password@host),
which should work without requiring the undocumented auth query parameter.

…y in RedisTrait (ckrack)

This PR was merged into the 6.4 branch.

Discussion
----------

[Cache] Fix DSN auth not passed to Redis/RedisCluster/Relay in RedisTrait

| Q             | A
| ------------- | ---
| Branch?       | 6.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        | Fix #62765 #63261
| License       | MIT

Populate $params["auth"] from DSN-parsed credentials for classes that
receive auth via their constructor or connect call (Redis, RedisCluster,
Relay), while explicitly excluding Sentinel mode where auth is applied
post-connect to the resolved master node (also using `Redis` or `Relay` classes).

This re-applies the fix from #62852 which was reverted in #63306 due to
a regression for Redis Sentinel users (#63261). The original fix
unconditionally set $params["auth"] from DSN credentials, which caused
them to be incorrectly passed to Sentinel servers that do not require
authentication.

As documented at https://symfony.com/doc/current/components/cache/adapters/redis_adapter.html#configure-the-connection,
the DSN supports password authentication (e.g. redis://user:password@host),
which should work without requiring the undocumented auth query parameter.

Commits
-------

62e4a4e [Cache] Fix DSN auth not passed to Redis/RedisCluster/Relay in RedisTrait

…y in RedisTrait (ckrack)

This PR was merged into the 6.4 branch.

Discussion
----------

[Cache] Fix DSN auth not passed to Redis/RedisCluster/Relay in RedisTrait

| Q             | A
| ------------- | ---
| Branch?       | 6.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        | Fix #62765 #63261
| License       | MIT

Populate $params["auth"] from DSN-parsed credentials for classes that
receive auth via their constructor or connect call (Redis, RedisCluster,
Relay), while explicitly excluding Sentinel mode where auth is applied
post-connect to the resolved master node (also using `Redis` or `Relay` classes).

This re-applies the fix from #62852 which was reverted in #63306 due to
a regression for Redis Sentinel users (#63261). The original fix
unconditionally set $params["auth"] from DSN credentials, which caused
them to be incorrectly passed to Sentinel servers that do not require
authentication.

As documented at https://symfony.com/doc/current/components/cache/adapters/redis_adapter.html#configure-the-connection,
the DSN supports password authentication (e.g. redis://user:password@host),
which should work without requiring the undocumented auth query parameter.

Commits
-------

467de8b [Cache] Fix DSN auth not passed to Redis/RedisCluster/Relay in RedisTrait