Merge pull request #10368 from 2403905/issue-10293

Disable download activity
owncloud · Oct 23, 2024 · 09dec76 · 09dec76
2 parents c6121f5 + ee42655
commit 09dec76
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 5 deletions.
diff --git a/changelog/unreleased/fix-disable-download-activity.md b/changelog/unreleased/fix-disable-download-activity.md
@@ -0,0 +1,6 @@
+Bugfix: Disable download activity
+
+We disable the download activity until we have a proper solution for it.
+
+https://github.com/owncloud/ocis/pull/10367
+https://github.com/owncloud/ocis/issues/10293
diff --git a/services/activitylog/pkg/service/service.go b/services/activitylog/pkg/service/service.go
@@ -103,11 +103,12 @@ func (a *ActivitylogService) Run() {
 			err = a.AddActivity(ev.FileRef, e.ID, utils.TSToTime(ev.Timestamp))
 		case events.FileTouched:
 			err = a.AddActivity(ev.Ref, e.ID, utils.TSToTime(ev.Timestamp))
-		case events.FileDownloaded:
-			// we are only interested in public link downloads - so no need to store others.
-			if ev.ImpersonatingUser.GetDisplayName() == "Public" {
-				err = a.AddActivity(ev.Ref, e.ID, utils.TSToTime(ev.Timestamp))
-			}
+		// Disabled https://github.com/owncloud/ocis/issues/10293
+		//case events.FileDownloaded:
+		// we are only interested in public link downloads - so no need to store others.
+		//if ev.ImpersonatingUser.GetDisplayName() == "Public" {
+		//	err = a.AddActivity(ev.Ref, e.ID, utils.TSToTime(ev.Timestamp))
+		//}
 		case events.ContainerCreated:
 			err = a.AddActivity(ev.Ref, e.ID, utils.TSToTime(ev.Timestamp))
 		case events.ItemTrashed: