Skip to content

Commit e1ddb5b

Browse files
committed
Do the scheme as part of build_cluster_connection_array
1 parent a8cf650 commit e1ddb5b

File tree

1 file changed

+23
-11
lines changed

1 file changed

+23
-11
lines changed

includes/class-predis.php

+23-11
Original file line numberDiff line numberDiff line change
@@ -86,11 +86,6 @@ public function connect( $read_timeout = null ) {
8686
$servers = $this->build_cluster_connection_array();
8787
$parameters['cluster'] = $servers;
8888
$options['cluster'] = 'redis';
89-
if ( defined('WP_REDIS_SCHEME') ) {
90-
$options['parameters'] = [
91-
'scheme' => WP_REDIS_SCHEME,
92-
];
93-
}
9489
}
9590

9691
if ( strcasecmp( 'unix', $parameters['scheme'] ) === 0 ) {
@@ -211,13 +206,30 @@ protected function build_cluster_connection_array() {
211206
$cluster = array_values( WP_REDIS_CLUSTER );
212207

213208
foreach ( $cluster as $key => $server ) {
214-
$connection_string = parse_url( $server );
209+
$components = parse_url( $server );
210+
211+
if ( ! empty( $components['scheme'] ) ) {
212+
$scheme = $components['scheme'];
213+
} elseif ( defined( 'WP_REDIS_SCHEME' ) ) {
214+
$scheme = WP_REDIS_SCHEME;
215+
} else {
216+
$scheme = null;
217+
}
215218

216-
$cluster[ $key ] = sprintf(
217-
"%s:%s",
218-
$connection_string['host'],
219-
$connection_string['port']
220-
);
219+
if ( isset( $scheme ) ) {
220+
$cluster[ $key ] = sprintf(
221+
'%s://%s:%d',
222+
$scheme,
223+
$components['host'],
224+
$components['port']
225+
);
226+
} else {
227+
$cluster[ $key ] = sprintf(
228+
'%s:%d',
229+
$components['host'],
230+
$components['port']
231+
);
232+
}
221233
}
222234

223235
return $cluster;

0 commit comments

Comments
 (0)