aboutsummaryrefslogtreecommitdiffhomepage
path: root/server/lib/job-queue
diff options
context:
space:
mode:
authorChocobozzz <me@florianbigard.com>2021-12-28 11:41:41 +0100
committerChocobozzz <me@florianbigard.com>2021-12-28 14:12:57 +0100
commit10a72a7e617273cdbe897766f77f427eb57b689a (patch)
treec467a690a6fbf4c70a75baf8ac0bd9db42bb5491 /server/lib/job-queue
parentf1569117f9bc468bc38b5b8d32df1bce40cc42ad (diff)
downloadPeerTube-10a72a7e617273cdbe897766f77f427eb57b689a.tar.gz
PeerTube-10a72a7e617273cdbe897766f77f427eb57b689a.tar.zst
PeerTube-10a72a7e617273cdbe897766f77f427eb57b689a.zip
Fix tests
Diffstat (limited to 'server/lib/job-queue')
-rw-r--r--server/lib/job-queue/handlers/activitypub-cleaner.ts38
1 files changed, 14 insertions, 24 deletions
diff --git a/server/lib/job-queue/handlers/activitypub-cleaner.ts b/server/lib/job-queue/handlers/activitypub-cleaner.ts
index 1540bf23a..509dd1cb5 100644
--- a/server/lib/job-queue/handlers/activitypub-cleaner.ts
+++ b/server/lib/job-queue/handlers/activitypub-cleaner.ts
@@ -14,29 +14,27 @@ import { VideoModel } from '@server/models/video/video'
14import { VideoCommentModel } from '@server/models/video/video-comment' 14import { VideoCommentModel } from '@server/models/video/video-comment'
15import { VideoShareModel } from '@server/models/video/video-share' 15import { VideoShareModel } from '@server/models/video/video-share'
16import { HttpStatusCode } from '@shared/models' 16import { HttpStatusCode } from '@shared/models'
17import { logger } from '../../../helpers/logger' 17import { logger, loggerTagsFactory } from '../../../helpers/logger'
18import { AccountVideoRateModel } from '../../../models/account/account-video-rate' 18import { AccountVideoRateModel } from '../../../models/account/account-video-rate'
19 19
20const lTags = loggerTagsFactory('ap-cleaner')
21
20// Job to clean remote interactions off local videos 22// Job to clean remote interactions off local videos
21 23
22async function processActivityPubCleaner (_job: Job) { 24async function processActivityPubCleaner (_job: Job) {
23 logger.info('Processing ActivityPub cleaner.') 25 logger.info('Processing ActivityPub cleaner.', lTags())
24 26
25 { 27 {
26 const rateUrls = await AccountVideoRateModel.listRemoteRateUrlsOfLocalVideos() 28 const rateUrls = await AccountVideoRateModel.listRemoteRateUrlsOfLocalVideos()
27 const { bodyValidator, deleter, updater } = rateOptionsFactory() 29 const { bodyValidator, deleter, updater } = rateOptionsFactory()
28 30
29 await map(rateUrls, async rateUrl => { 31 await map(rateUrls, async rateUrl => {
30 try { 32 const result = await updateObjectIfNeeded({ url: rateUrl, bodyValidator, updater, deleter })
31 const result = await updateObjectIfNeeded({ url: rateUrl, bodyValidator, updater, deleter })
32 33
33 if (result?.status === 'deleted') { 34 if (result?.status === 'deleted') {
34 const { videoId, type } = result.data 35 const { videoId, type } = result.data
35 36
36 await VideoModel.updateRatesOf(videoId, type, undefined) 37 await VideoModel.updateRatesOf(videoId, type, undefined)
37 }
38 } catch (err) {
39 logger.warn('Cannot update/delete remote AP rate %s.', rateUrl, { err })
40 } 38 }
41 }, { concurrency: AP_CLEANER.CONCURRENCY }) 39 }, { concurrency: AP_CLEANER.CONCURRENCY })
42 } 40 }
@@ -46,11 +44,7 @@ async function processActivityPubCleaner (_job: Job) {
46 const { bodyValidator, deleter, updater } = shareOptionsFactory() 44 const { bodyValidator, deleter, updater } = shareOptionsFactory()
47 45
48 await map(shareUrls, async shareUrl => { 46 await map(shareUrls, async shareUrl => {
49 try { 47 await updateObjectIfNeeded({ url: shareUrl, bodyValidator, updater, deleter })
50 await updateObjectIfNeeded({ url: shareUrl, bodyValidator, updater, deleter })
51 } catch (err) {
52 logger.warn('Cannot update/delete remote AP share %s.', shareUrl, { err })
53 }
54 }, { concurrency: AP_CLEANER.CONCURRENCY }) 48 }, { concurrency: AP_CLEANER.CONCURRENCY })
55 } 49 }
56 50
@@ -59,11 +53,7 @@ async function processActivityPubCleaner (_job: Job) {
59 const { bodyValidator, deleter, updater } = commentOptionsFactory() 53 const { bodyValidator, deleter, updater } = commentOptionsFactory()
60 54
61 await map(commentUrls, async commentUrl => { 55 await map(commentUrls, async commentUrl => {
62 try { 56 await updateObjectIfNeeded({ url: commentUrl, bodyValidator, updater, deleter })
63 await updateObjectIfNeeded({ url: commentUrl, bodyValidator, updater, deleter })
64 } catch (err) {
65 logger.warn('Cannot update/delete remote AP comment %s.', commentUrl, { err })
66 }
67 }, { concurrency: AP_CLEANER.CONCURRENCY }) 57 }, { concurrency: AP_CLEANER.CONCURRENCY })
68 } 58 }
69} 59}
@@ -85,7 +75,7 @@ async function updateObjectIfNeeded <T> (options: {
85 const { url, bodyValidator, updater, deleter } = options 75 const { url, bodyValidator, updater, deleter } = options
86 76
87 const on404OrTombstone = async () => { 77 const on404OrTombstone = async () => {
88 logger.info('Removing remote AP object %s.', url) 78 logger.info('Removing remote AP object %s.', url, lTags(url))
89 const data = await deleter(url) 79 const data = await deleter(url)
90 80
91 return { status: 'deleted' as 'deleted', data } 81 return { status: 'deleted' as 'deleted', data }
@@ -107,7 +97,7 @@ async function updateObjectIfNeeded <T> (options: {
107 throw new Error(`New url ${newUrl} has not the same host than old url ${url}`) 97 throw new Error(`New url ${newUrl} has not the same host than old url ${url}`)
108 } 98 }
109 99
110 logger.info('Updating remote AP object %s.', url) 100 logger.info('Updating remote AP object %s.', url, lTags(url))
111 const data = await updater(url, newUrl) 101 const data = await updater(url, newUrl)
112 102
113 return { status: 'updated', data } 103 return { status: 'updated', data }
@@ -120,11 +110,11 @@ async function updateObjectIfNeeded <T> (options: {
120 return on404OrTombstone() 110 return on404OrTombstone()
121 } 111 }
122 112
123 logger.debug('Remote AP object %s is unavailable.', url) 113 logger.debug('Remote AP object %s is unavailable.', url, lTags(url))
124 114
125 const unavailability = await Redis.Instance.addAPUnavailability(url) 115 const unavailability = await Redis.Instance.addAPUnavailability(url)
126 if (unavailability >= AP_CLEANER.UNAVAILABLE_TRESHOLD) { 116 if (unavailability >= AP_CLEANER.UNAVAILABLE_TRESHOLD) {
127 logger.info('Removing unavailable AP resource %s.', url) 117 logger.info('Removing unavailable AP resource %s.', url, lTags(url))
128 return on404OrTombstone() 118 return on404OrTombstone()
129 } 119 }
130 120