Skip to content

Commit f9f7067

Browse files
author
Artem
committed
#RI-4030 add telemetry events on edit
1 parent 8370216 commit f9f7067

File tree

2 files changed

+13
-3
lines changed

2 files changed

+13
-3
lines changed

redisinsight/api/src/modules/database/database.analytics.spec.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -193,12 +193,16 @@ describe('DatabaseAnalytics', () => {
193193
useTLS: 'disabled',
194194
verifyTLSCertificate: 'disabled',
195195
useTLSAuthClients: 'disabled',
196+
useSNI: 'enabled',
197+
useSSH: 'disabled',
196198
previousValues: {
197199
connectionType: prev.connectionType,
198200
provider: prev.provider,
199201
useTLS: 'enabled',
200202
verifyTLSCertificate: 'enabled',
201203
useTLSAuthClients: 'enabled',
204+
useSNI: 'enabled',
205+
useSSH: 'disabled',
202206
},
203207
},
204208
);
@@ -223,10 +227,14 @@ describe('DatabaseAnalytics', () => {
223227
useTLS: 'enabled',
224228
verifyTLSCertificate: 'enabled',
225229
useTLSAuthClients: 'enabled',
230+
useSNI: 'enabled',
231+
useSSH: 'disabled',
226232
previousValues: {
227233
connectionType: prev.connectionType,
228234
provider: prev.provider,
229235
useTLS: 'disabled',
236+
useSNI: 'disabled',
237+
useSSH: 'disabled',
230238
verifyTLSCertificate: 'disabled',
231239
useTLSAuthClients: 'disabled',
232240
},

redisinsight/api/src/modules/database/database.analytics.ts

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -57,9 +57,7 @@ export class DatabaseAnalytics extends TelemetryBaseService {
5757
useTLSAuthClients: instance?.clientCert
5858
? 'enabled'
5959
: 'disabled',
60-
useSNI: instance?.tlsServername
61-
? 'enabled'
62-
: 'disabled',
60+
useSNI: instance?.tlsServername ? 'enabled' : 'disabled',
6361
useSSH: instance?.ssh ? 'enabled' : 'disabled',
6462
version: additionalInfo?.version,
6563
numberOfKeys: additionalInfo?.totalKeys,
@@ -97,10 +95,14 @@ export class DatabaseAnalytics extends TelemetryBaseService {
9795
? 'enabled'
9896
: 'disabled',
9997
useTLSAuthClients: cur?.clientCert ? 'enabled' : 'disabled',
98+
useSNI: cur?.tlsServername ? 'enabled' : 'disabled',
99+
useSSH: cur?.ssh ? 'enabled' : 'disabled',
100100
previousValues: {
101101
connectionType: prev.connectionType,
102102
provider: prev.provider,
103103
useTLS: prev.tls ? 'enabled' : 'disabled',
104+
useSNI: prev?.tlsServername ? 'enabled' : 'disabled',
105+
useSSH: prev?.ssh ? 'enabled' : 'disabled',
104106
verifyTLSCertificate: prev?.verifyServerCert
105107
? 'enabled'
106108
: 'disabled',

0 commit comments

Comments
 (0)