From 205fdcba424ac89173bd05b6faec5cabce7d8d7e Mon Sep 17 00:00:00 2001 From: Alexander Date: Sat, 24 Dec 2022 20:53:11 +0100 Subject: [PATCH] fix(backend/clickhouse): fixed batch insert panic because of wrong error source (#900) --- ee/backend/pkg/db/clickhouse/connector.go | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/ee/backend/pkg/db/clickhouse/connector.go b/ee/backend/pkg/db/clickhouse/connector.go index de94fedc2..20996c073 100644 --- a/ee/backend/pkg/db/clickhouse/connector.go +++ b/ee/backend/pkg/db/clickhouse/connector.go @@ -289,7 +289,13 @@ func (c *connectorImpl) InsertWebErrorEvent(session *types.Session, msg *types.E keys = append(keys, k) values = append(values, v) } - + // Check error source before insert to avoid panic from clickhouse lib + switch msg.Source { + case "js_exception", "bugsnag", "cloudwatch", "datadog", "elasticsearch", "newrelic", "rollbar", "sentry", "stackdriver", "sumologic": + default: + return fmt.Errorf("unknown error source: %s", msg.Source) + } + // Insert event to batch if err := c.batches["errors"].Append( session.SessionID, uint16(session.ProjectID),