Skip to content

Commit 48cda7c

Browse files
committed
Prefer enable_tls for sentinel config consistency
1 parent 9c403ed commit 48cda7c

3 files changed

Lines changed: 19 additions & 17 deletions

File tree

config/default.yaml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -95,16 +95,17 @@ redis:
9595
port: 6379
9696
auth: null
9797
db: 0
98-
tls: false
99-
tls_settings:
98+
enable_tls: false
99+
tls_settings:
100100
reject_unauthorized: false
101101
ca: '/absolute/path/to/server-certificates/root.crt'
102102
cert: '/absolute/path/to/client-certificates/postgresql.crt'
103103
key: '/absolute/path/to/client-key/postgresql.key'
104+
104105
sentinel:
105106
enabled: false
106107
enable_tls: false
107-
tls_settings:
108+
tls_settings:
108109
reject_unauthorized: false
109110
ca: '/absolute/path/to/server-certificates/root.crt'
110111
cert: '/absolute/path/to/client-certificates/postgresql.crt'

config/production.yaml.example

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -93,16 +93,17 @@ redis:
9393
port: 6379
9494
auth: null
9595
db: 0
96-
tls: false
97-
tls_settings:
96+
enable_tls: false
97+
tls_settings:
9898
reject_unauthorized: false
9999
ca: '/absolute/path/to/server-certificates/root.crt'
100100
cert: '/absolute/path/to/client-certificates/postgresql.crt'
101101
key: '/absolute/path/to/client-key/postgresql.key'
102+
102103
sentinel:
103104
enabled: false
104105
enable_tls: false
105-
tls_settings:
106+
tls_settings:
106107
reject_unauthorized: false
107108
ca: '/absolute/path/to/server-certificates/root.crt'
108109
cert: '/absolute/path/to/client-certificates/postgresql.crt'

server/core/initializers/config.ts

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -51,20 +51,20 @@ const CONFIG = {
5151
SOCKET: config.has('redis.socket') ? config.get<string>('redis.socket') : null,
5252
AUTH: config.has('redis.auth') ? config.get<string>('redis.auth') : null,
5353
DB: config.has('redis.db') ? config.get<number>('redis.db') : null,
54-
TLS: config.has('redis.tls') ? config.get<boolean>('redis.tls') : false,
54+
TLS: config.has('redis.enable_tls') ? config.get<boolean>('redis.enable_tls') : false,
5555
TLS_SETTINGS: {
5656
get REJECT_UNAUTHORIZED () {
57-
return config.has('redis.tls_settings.reject_unauthorized')
58-
? config.get<boolean>('redis.tls_settings.reject_unauthorized')
57+
return config.has('redis.tls_settings.reject_unauthorized')
58+
? config.get<boolean>('redis.tls_settings.reject_unauthorized')
5959
: false
6060
},
61-
get CA() {
61+
get CA () {
6262
return config.has('redis.tls_settings.ca') ? config.get<string>('redis.tls_settings.ca') : null
6363
},
64-
get CERT() {
64+
get CERT () {
6565
return config.has('redis.tls_settings.cert') ? config.get<string>('redis.tls_settings.cert') : null
6666
},
67-
get KEY() {
67+
get KEY () {
6868
return config.has('redis.tls_settings.key') ? config.get<string>('redis.tls_settings.key') : null
6969
}
7070
},
@@ -73,17 +73,17 @@ const CONFIG = {
7373
ENABLE_TLS: config.has('redis.sentinel.enable_tls') ? config.get<boolean>('redis.sentinel.enable_tls') : false,
7474
TLS_SETTINGS: {
7575
get REJECT_UNAUTHORIZED () {
76-
return config.has('redis.sentinel.tls_settings.reject_unauthorized')
77-
? config.get<boolean>('redis.sentinel.tls_settings.reject_unauthorized')
76+
return config.has('redis.sentinel.tls_settings.reject_unauthorized')
77+
? config.get<boolean>('redis.sentinel.tls_settings.reject_unauthorized')
7878
: false
7979
},
80-
get CA() {
80+
get CA () {
8181
return config.has('redis.sentinel.tls_settings.ca') ? config.get<string>('redis.sentinel.tls_settings.ca') : null
8282
},
83-
get CERT() {
83+
get CERT () {
8484
return config.has('redis.sentinel.tls_settings.cert') ? config.get<string>('redis.sentinel.tls_settings.cert') : null
8585
},
86-
get KEY() {
86+
get KEY () {
8787
return config.has('redis.sentinel.tls_settings.key') ? config.get<string>('redis.sentinel.tls_settings.key') : null
8888
}
8989
},

0 commit comments

Comments
 (0)