summaryrefslogtreecommitdiff
path: root/packages/backend/src
diff options
context:
space:
mode:
authorHazelnoot <acomputerdog@gmail.com>2025-03-15 20:46:28 +0000
committerHazelnoot <acomputerdog@gmail.com>2025-03-15 20:46:28 +0000
commite3d949ced6115a3e16c7f7dcd3bddd9ca4841469 (patch)
treeb104c60145a1ef015edfab8293b603c351c1c5a6 /packages/backend/src
parentmerge: Add external url warning to url previews (!944) (diff)
parentfix indent (diff)
downloadsharkey-e3d949ced6115a3e16c7f7dcd3bddd9ca4841469.tar.gz
sharkey-e3d949ced6115a3e16c7f7dcd3bddd9ca4841469.tar.bz2
sharkey-e3d949ced6115a3e16c7f7dcd3bddd9ca4841469.zip
merge: fetch linked notes manually, unless we have them in DB - fixes 1006 (!945)
View MR for information: https://activitypub.software/TransFem-org/Sharkey/-/merge_requests/945 Closes #1006 Approved-by: Marie <github@yuugi.dev> Approved-by: Hazelnoot <acomputerdog@gmail.com>
Diffstat (limited to 'packages/backend/src')
-rw-r--r--packages/backend/src/server/web/UrlPreviewService.ts14
1 files changed, 12 insertions, 2 deletions
diff --git a/packages/backend/src/server/web/UrlPreviewService.ts b/packages/backend/src/server/web/UrlPreviewService.ts
index 19dac1dfb8..cf130a21ec 100644
--- a/packages/backend/src/server/web/UrlPreviewService.ts
+++ b/packages/backend/src/server/web/UrlPreviewService.ts
@@ -19,6 +19,7 @@ import { MiMeta } from '@/models/Meta.js';
import { RedisKVCache } from '@/misc/cache.js';
import { UtilityService } from '@/core/UtilityService.js';
import type { FastifyRequest, FastifyReply } from 'fastify';
+import { ApDbResolverService } from '@/core/activitypub/ApDbResolverService.js';
@Injectable()
export class UrlPreviewService {
@@ -38,6 +39,7 @@ export class UrlPreviewService {
private httpRequestService: HttpRequestService,
private loggerService: LoggerService,
private utilityService: UtilityService,
+ private apDbResolverService: ApDbResolverService,
) {
this.logger = this.loggerService.getLogger('url-preview');
this.previewCache = new RedisKVCache<SummalyResult>(this.redisClient, 'summaly', {
@@ -102,12 +104,16 @@ export class UrlPreviewService {
}
const key = `${url}@${lang}`;
- const cached = await this.previewCache.get(key);
+ const cached = await this.previewCache.get(key) as SummalyResult & { haveNoteLocally?: boolean };
if (cached !== undefined) {
this.logger.info(`Returning cache preview of ${key}`);
// Cache 7days
reply.header('Cache-Control', 'max-age=604800, immutable');
+ if (cached.activityPub) {
+ cached.haveNoteLocally = !! await this.apDbResolverService.getNoteFromApId(cached.activityPub);
+ }
+
return cached;
}
@@ -116,7 +122,7 @@ export class UrlPreviewService {
: `Getting preview of ${key} ...`);
try {
- const summary = this.meta.urlPreviewSummaryProxyUrl
+ const summary: SummalyResult & { haveNoteLocally?: boolean } = this.meta.urlPreviewSummaryProxyUrl
? await this.fetchSummaryFromProxy(url, this.meta, lang)
: await this.fetchSummary(url, this.meta, lang);
@@ -135,6 +141,10 @@ export class UrlPreviewService {
this.previewCache.set(key, summary);
+ if (summary.activityPub) {
+ summary.haveNoteLocally = !! await this.apDbResolverService.getNoteFromApId(summary.activityPub);
+ }
+
// Cache 7days
reply.header('Cache-Control', 'max-age=604800, immutable');