fix: tests

e2e/src/api/specs/user.e2e-spec.ts

@@ -42,7 +42,9 @@ describe('/user', () => {
     });
 
     it('should get users', async () => {
-      const { status, body } = await request(app).get('/user').set('Authorization', `Bearer ${admin.accessToken}`);
+      const { status, body } = await request(app)
+        .get('/user/admin')
+        .set('Authorization', `Bearer ${admin.accessToken}`);
       expect(status).toEqual(200);
       expect(body).toHaveLength(5);
       expect(body).toEqual(
@@ -288,7 +290,6 @@ describe('/user', () => {
       expect(status).toBe(200);
       expect(body).toEqual({
         ...before,
-        updatedAt: expect.any(String),
         name: 'Name',
       });
       expect(before.updatedAt).not.toEqual(body.updatedAt);

open-api/immich-openapi-specs.json

@@ -6844,7 +6844,7 @@
         ]
       }
     },
-    "/user/info/{id}": {
+    "/user/admin/info/{id}": {
       "get": {
         "operationId": "getUserById",
         "parameters": [
@@ -6858,6 +6858,48 @@
             }
           }
         ],
+        "responses": {
+          "200": {
+            "content": {
+              "application/json": {
+                "schema": {
+                  "$ref": "#/components/schemas/UserResponseDto"
+                }
+              }
+            },
+            "description": ""
+          }
+        },
+        "security": [
+          {
+            "bearer": []
+          },
+          {
+            "cookie": []
+          },
+          {
+            "api_key": []
+          }
+        ],
+        "tags": [
+          "User"
+        ]
+      }
+    },
+    "/user/info/{id}": {
+      "get": {
+        "operationId": "getPublicUserById",
+        "parameters": [
+          {
+            "name": "id",
+            "required": true,
+            "in": "path",
+            "schema": {
+              "format": "uuid",
+              "type": "string"
+            }
+          }
+        ],
         "responses": {
           "200": {
             "content": {

open-api/typescript-sdk/src/fetch-client.ts

@@ -2832,6 +2832,16 @@ export function getAllUsers({ isAll }: {
 }
 export function getUserById({ id }: {
     id: string;
+}, opts?: Oazapfts.RequestOpts) {
+    return oazapfts.ok(oazapfts.fetchJson<{
+        status: 200;
+        data: UserResponseDto;
+    }>(`/user/admin/info/${encodeURIComponent(id)}`, {
+        ...opts
+    }));
+}
+export function getPublicUserById({ id }: {
+    id: string;
 }, opts?: Oazapfts.RequestOpts) {
     return oazapfts.ok(oazapfts.fetchJson<{
         status: 200;

server/src/controllers/user.controller.ts

@@ -32,18 +32,24 @@ export class UserController {
   constructor(private service: UserService) {}
 
   @Get()
-  getAllPublicUsers(@Auth() auth: AuthDto): Promise<UserDto[]> {
-    return this.service.getAllPublic(auth);
+  getAllPublicUsers(): Promise<UserDto[]> {
+    return this.service.getAllPublic();
   }
 
   @AdminRoute()
   @Get('admin')
-  getAllUsers(@Auth() auth: AuthDto, @Query('isAll') isAll: boolean): Promise<UserResponseDto[]> {
-    return this.service.getAll(auth, isAll);
+  getAllUsers(@Query('isAll') isAll: boolean): Promise<UserResponseDto[]> {
+    return this.service.getAll(isAll);
   }
 
   @Get('info/:id')
-  getUserById(@Param() { id }: UUIDParamDto): Promise<UserDto> {
+  getPublicUserById(@Param() { id }: UUIDParamDto): Promise<UserDto> {
+    return this.service.getPublic(id);
+  }
+
+  @AdminRoute()
+  @Get('admin/info/:id')
+  getUserById(@Param() { id }: UUIDParamDto): Promise<UserResponseDto> {
     return this.service.get(id);
   }
 

server/src/services/user.service.ts

@@ -50,16 +50,25 @@ export class UserService {
     return users.map((user) => mapUser(user));
   }
 
-  async getAll(auth: AuthDto, isAll: boolean): Promise<UserResponseDto[]> {
+  async getAll(isAll: boolean): Promise<UserResponseDto[]> {
     const users = await this.userRepository.getList({ withDeleted: !isAll });
     return users.map((user) => mapUser(user));
   }
 
-  async getAllPublic(auth: AuthDto): Promise<UserDto[] | UserResponseDto[]> {
+  async getAllPublic(): Promise<UserDto[] | UserResponseDto[]> {
     const users = await this.userRepository.getList({ withDeleted: false });
     return users.map((user) => mapSimpleUser(user));
   }
 
+  async getPublic(userId: string): Promise<UserDto> {
+    const user = await this.userRepository.get(userId, { withDeleted: false });
+    if (!user) {
+      throw new NotFoundException('User not found');
+    }
+
+    return mapSimpleUser(user);
+  }
+
   async get(userId: string): Promise<UserResponseDto> {
     const user = await this.userRepository.get(userId, { withDeleted: false });
     if (!user) {

web/src/routes/(user)/partners/[userId]/+page.ts

@@ -1,11 +1,11 @@
 import { authenticate } from '$lib/utils/auth';
-import { getUserById } from '@immich/sdk';
+import { getPublicUserById } from '@immich/sdk';
 import type { PageLoad } from './$types';
 
 export const load = (async ({ params }) => {
   await authenticate();
 
-  const partner = await getUserById({ id: params.userId });
+  const partner = await getPublicUserById({ id: params.userId });
 
   return {
     partner,

web/src/routes/admin/library-management/+page.svelte

@@ -23,14 +23,14 @@
     deleteLibrary,
     getAllLibraries,
     getLibraryStatistics,
-    getUserById,
     LibraryType,
     removeOfflineFiles,
     scanLibrary,
     updateLibrary,
     type LibraryResponseDto,
     type LibraryStatsResponseDto,
-    type UserResponseDto,
+    type UserDto,
+    getPublicUserById,
   } from '@immich/sdk';
   import { mdiDatabase, mdiDotsVertical, mdiPlusBoxOutline, mdiSync, mdiUpload } from '@mdi/js';
   import { onMount } from 'svelte';
@@ -43,7 +43,7 @@
   let libraries: LibraryResponseDto[] = [];
 
   let stats: LibraryStatsResponseDto[] = [];
-  let owner: UserResponseDto[] = [];
+  let owner: UserDto[] = [];
   let photos: number[] = [];
   let videos: number[] = [];
   let totalCount: number[] = [];
@@ -99,7 +99,7 @@
 
   const refreshStats = async (listIndex: number) => {
     stats[listIndex] = await getLibraryStatistics({ id: libraries[listIndex].id });
-    owner[listIndex] = await getUserById({ id: libraries[listIndex].ownerId });
+    owner[listIndex] = await getPublicUserById({ id: libraries[listIndex].ownerId });
     photos[listIndex] = stats[listIndex].photos;
     videos[listIndex] = stats[listIndex].videos;
     totalCount[listIndex] = stats[listIndex].total;