Browse Source

Fix log suppresion issue (#1887)

The check for log suppresion enables archive scanning by default with no
max depth.
bplaxco 8 tháng trước cách đây
mục cha
commit
2463f11e71
1 tập tin đã thay đổi với 9 bổ sung6 xóa
  1. 9 6
      sources/file.go

+ 9 - 6
sources/file.go

@@ -55,12 +55,15 @@ func (s *File) Fragments(ctx context.Context, yield FragmentsFunc) error {
 	// and fall back on treating it like a normal file and let fileFragments
 	// decide what to do with it.
 	if err == nil && format != nil {
-		if (s.archiveDepth+1 > s.MaxArchiveDepth) && s.MaxArchiveDepth != 0 {
-			logging.Warn().Str(
-				"path", s.FullPath(),
-			).Int(
-				"max_archive_depth", s.MaxArchiveDepth,
-			).Msg("skipping archive: exceeds max archive depth")
+		if s.archiveDepth+1 > s.MaxArchiveDepth {
+			// Only warn when the feature is enabled
+			if s.MaxArchiveDepth != 0 {
+				logging.Warn().Str(
+					"path", s.FullPath(),
+				).Int(
+					"max_archive_depth", s.MaxArchiveDepth,
+				).Msg("skipping archive: exceeds max archive depth")
+			}
 			return nil
 		}
 		if extractor, ok := format.(archives.Extractor); ok {