fix(server): asset count on meta tag of shared links (#9107)

* fix asset count on meta tag of shared links

* fix lint

* fix style

* add e2e tests

e2e/src/api/specs/shared-link.e2e-spec.ts

@@ -9,7 +9,7 @@ import {
 } from '@immich/sdk';
 import { createUserDto, uuidDto } from 'src/fixtures';
 import { errorDto } from 'src/responses';
-import { app, asBearerAuth, utils } from 'src/utils';
+import { app, asBearerAuth, shareUrl, utils } from 'src/utils';
 import request from 'supertest';
 import { beforeAll, describe, expect, it } from 'vitest';
 
@@ -89,6 +89,31 @@ describe('/shared-link', () => {
     await deleteUser({ id: user2.userId, deleteUserDto: {} }, { headers: asBearerAuth(admin.accessToken) });
   });
 
+  describe('GET /share/${key}', () => {
+    it('should have correct asset count in meta tag for non-empty album', async () => {
+      const resp = await request(shareUrl).get(`/${linkWithMetadata.key}`);
+      expect(resp.status).toBe(200);
+      expect(resp.header['content-type']).toContain('text/html');
+      expect(resp.text).toContain(
+        `<meta name="description" content="${metadataAlbum.assets.length} shared photos & videos" />`,
+      );
+    });
+
+    it('should have correct asset count in meta tag for empty album', async () => {
+      const resp = await request(shareUrl).get(`/${linkWithAlbum.key}`);
+      expect(resp.status).toBe(200);
+      expect(resp.header['content-type']).toContain('text/html');
+      expect(resp.text).toContain(`<meta name="description" content="0 shared photos & videos" />`);
+    });
+
+    it('should have correct asset count in meta tag for shared asset', async () => {
+      const resp = await request(shareUrl).get(`/${linkWithAssets.key}`);
+      expect(resp.status).toBe(200);
+      expect(resp.header['content-type']).toContain('text/html');
+      expect(resp.text).toContain(`<meta name="description" content="1 shared photos & videos" />`);
+    });
+  });
+
   describe('GET /shared-link', () => {
     it('should require authentication', async () => {
       const { status, body } = await request(app).get('/shared-link');

e2e/src/utils.ts

@@ -53,6 +53,7 @@ type AssetData = { bytes?: Buffer; filename: string };
 const dbUrl = 'postgres://postgres:postgres@127.0.0.1:5433/immich';
 const baseUrl = 'http://127.0.0.1:2283';
 
+export const shareUrl = `${baseUrl}/share`;
 export const app = `${baseUrl}/api`;
 // TODO move test assets into e2e/assets
 export const testAssetDir = path.resolve(`./../server/test/assets/`);

server/src/services/shared-link.service.ts

@@ -185,7 +185,7 @@ export class SharedLinkService {
 
     const sharedLink = await this.findOrFail(auth.sharedLink.userId, auth.sharedLink.id);
     const assetId = sharedLink.album?.albumThumbnailAssetId || sharedLink.assets[0]?.id;
-    const assetCount = sharedLink.assets.length ?? sharedLink.album?.assets.length ?? 0;
+    const assetCount = sharedLink.assets.length > 0 ? sharedLink.assets.length : sharedLink.album?.assets.length || 0;
 
     return {
       title: sharedLink.album ? sharedLink.album.albumName : 'Public Share',