Skip to content

Commit 05a926d

Browse files
authored
fix: handle missing eventUpload config gracefully (#5898)
# Description handle missing eventUpload config gracefully ## Linear Ticket - ## Security - [x] The code changed/added as part of this pull request won't create any security issues with how the software is being used.
1 parent 39f7e75 commit 05a926d

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

services/debugger/source/eventUploader.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ package sourcedebugger
33
//go:generate mockgen -destination=./mocks/mock.go -package=mocks github.com/rudderlabs/rudder-server/services/debugger/source SourceDebugger
44
import (
55
"context"
6+
"errors"
67
"fmt"
78
"slices"
89
"sync"
@@ -137,7 +138,7 @@ func (h *Handle) updateConfig(config map[string]backendconfig.ConfigT) {
137138
for _, source := range wConfig.Sources {
138139
if source.Config != nil {
139140
eventUploadEnabled, err := jsonparser.GetBoolean(source.Config, "eventUpload")
140-
if err != nil {
141+
if err != nil && !errors.Is(err, jsonparser.KeyPathNotFoundError) && !errors.Is(err, jsonparser.NullValueError) {
141142
h.log.Errorn("error while parsing eventUpload config", obskit.SourceID(source.ID), obskit.Error(err))
142143
}
143144
if source.Enabled && eventUploadEnabled {

0 commit comments

Comments
 (0)