Skip to content

Commit a37456e

Browse files
authored
Merge pull request #1744 from RedisInsight/be/bugfix/RI-4177_remove_port_from_telemetry
#RI-4177 - remove port from telemetry event
2 parents 3e0e5f5 + ae2da12 commit a37456e

File tree

2 files changed

+0
-6
lines changed

2 files changed

+0
-6
lines changed

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

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -190,7 +190,6 @@ describe('DatabaseAnalytics', () => {
190190
expect(sendEventSpy).toHaveBeenCalledWith(
191191
TelemetryEvents.RedisInstanceEditedByUser,
192192
{
193-
port: cur.port,
194193
databaseId: cur.id,
195194
connectionType: cur.connectionType,
196195
provider: HostingProvider.RE_CLUSTER,
@@ -201,7 +200,6 @@ describe('DatabaseAnalytics', () => {
201200
useSSH: 'disabled',
202201
timeout: mockDatabaseWithTlsAuth.timeout / 1_000, // milliseconds to seconds
203202
previousValues: {
204-
port: prev.port,
205203
connectionType: prev.connectionType,
206204
provider: prev.provider,
207205
useTLS: 'enabled',
@@ -227,7 +225,6 @@ describe('DatabaseAnalytics', () => {
227225
expect(sendEventSpy).toHaveBeenCalledWith(
228226
TelemetryEvents.RedisInstanceEditedByUser,
229227
{
230-
port: cur.port,
231228
databaseId: cur.id,
232229
connectionType: cur.connectionType,
233230
provider: HostingProvider.RE_CLUSTER,
@@ -238,7 +235,6 @@ describe('DatabaseAnalytics', () => {
238235
useSSH: 'disabled',
239236
timeout: mockDatabaseWithTlsAuth.timeout / 1_000, // milliseconds to seconds
240237
previousValues: {
241-
port: prev.port,
242238
connectionType: prev.connectionType,
243239
provider: prev.provider,
244240
useTLS: 'disabled',

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

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,6 @@ export class DatabaseAnalytics extends TelemetryBaseService {
8888
this.sendEvent(
8989
TelemetryEvents.RedisInstanceEditedByUser,
9090
{
91-
port: cur.port,
9291
databaseId: cur.id,
9392
connectionType: cur.connectionType,
9493
provider: cur.provider,
@@ -101,7 +100,6 @@ export class DatabaseAnalytics extends TelemetryBaseService {
101100
useSSH: cur?.ssh ? 'enabled' : 'disabled',
102101
timeout: cur?.timeout / 1_000, // milliseconds to seconds
103102
previousValues: {
104-
port: prev.port,
105103
connectionType: prev.connectionType,
106104
provider: prev.provider,
107105
useTLS: prev.tls ? 'enabled' : 'disabled',

0 commit comments

Comments
 (0)