fix: More accurate logging (#1243)

himank · web-flow · commit 869b65542961 · 2023-05-29T19:58:02.000-07:00
diff --git a/server/services/v1/database/search_runner.go b/server/services/v1/database/search_runner.go
@@ -65,12 +65,13 @@ func (runner *SearchQueryRunner) ReadOnly(ctx context.Context, tenant *metadata.
 	if err != nil {
 		return Response{}, ctx, err
 	}
-	if config.DefaultConfig.Search.LogFilter {
+	if config.DefaultConfig.Search.LogFilter && !wrappedF.None() {
 		log.Error().
 			Str("tenant", tenant.Name()).
 			Str("project", db.Name()).
 			Str("collection", collection.Name).
 			Str("filter", string(runner.req.Filter)).
+			Bool("search_index", wrappedF.IsSearchIndexed()).
 			Msg("collection search filters")
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -65,12 +65,13 @@ func (runner SearchQueryRunner) ReadOnly(ctx context.Context, tenant metadata.`
`65`	`65`	`if err != nil {`
`66`	`66`	`return Response{}, ctx, err`
`67`	`67`	`}`
`68`		`- if config.DefaultConfig.Search.LogFilter {`
	`68`	`+ if config.DefaultConfig.Search.LogFilter && !wrappedF.None() {`
`69`	`69`	`log.Error().`
`70`	`70`	`Str("tenant", tenant.Name()).`
`71`	`71`	`Str("project", db.Name()).`
`72`	`72`	`Str("collection", collection.Name).`
`73`	`73`	`Str("filter", string(runner.req.Filter)).`
	`74`	`+ Bool("search_index", wrappedF.IsSearchIndexed()).`
`74`	`75`	`Msg("collection search filters")`
`75`	`76`	`}`
`76`	`77`