Skip to content

Commit 9b4506b

Browse files
committed
remove appsec segfault
Signed-off-by: Eliott Bouhana <[email protected]>
1 parent ea8d4bb commit 9b4506b

File tree

2 files changed

+5
-9
lines changed

2 files changed

+5
-9
lines changed

internal/appsec/config/config.go

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -19,25 +19,20 @@ import (
1919
)
2020

2121
func init() {
22-
registerAppConfigTelemetry()
23-
}
24-
25-
// Register the global app telemetry configuration.
26-
func registerAppConfigTelemetry() {
27-
registerSCAAppConfigTelemetry(telemetry.GlobalClient())
22+
registerSCAAppConfigTelemetry()
2823
}
2924

3025
// Register the global app telemetry configuration related to the Software Composition Analysis (SCA) product.
3126
// Report over telemetry whether SCA's enablement env var was set or not along with its value. Nothing is reported in
3227
// case of an error or if the env var is not set.
33-
func registerSCAAppConfigTelemetry(client telemetry.Client) {
28+
func registerSCAAppConfigTelemetry() {
3429
val, defined, err := parseBoolEnvVar(EnvSCAEnabled)
3530
if err != nil {
3631
log.Error("appsec: %v", err)
3732
return
3833
}
3934
if defined {
40-
client.RegisterAppConfig(EnvSCAEnabled, val, telemetry.OriginEnvVar)
35+
telemetry.RegisterAppConfig(EnvSCAEnabled, val, telemetry.OriginEnvVar)
4136
}
4237
}
4338

internal/appsec/config/config_test.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,8 +51,9 @@ func TestSCAEnabled(t *testing.T) {
5151

5252
telemetryClient := new(telemetrytest.MockClient)
5353
telemetryClient.On("RegisterAppConfig", EnvSCAEnabled, tc.expectedValue, telemetry.OriginEnvVar).Return()
54+
defer telemetry.MockClient(telemetryClient)()
5455

55-
registerSCAAppConfigTelemetry(telemetryClient)
56+
registerSCAAppConfigTelemetry()
5657

5758
if tc.telemetryExpected {
5859
telemetryClient.AssertCalled(t, "RegisterAppConfig", EnvSCAEnabled, tc.expectedValue, telemetry.OriginEnvVar)

0 commit comments

Comments
 (0)