fix(server): OAuth quota size

Author: wuzihao051119

i18n/en.json

@@ -207,7 +207,7 @@
     "oauth_storage_quota_claim": "Storage quota claim",
     "oauth_storage_quota_claim_description": "Automatically set the user's storage quota to the value of this claim.",
     "oauth_storage_quota_default": "Default storage quota (GiB)",
-    "oauth_storage_quota_default_description": "Quota in GiB to be used when no claim is provided (Enter 0 for unlimited quota).",
+    "oauth_storage_quota_default_description": "Quota in GiB to be used when no claim is provided.",
     "oauth_timeout": "Request Timeout",
     "oauth_timeout_description": "Timeout for requests in milliseconds",
     "offline_paths": "Offline Paths",

mobile/openapi/lib/model/system_config_o_auth_dto.dart

@@ -13685,8 +13685,9 @@
             "type": "string"
           },
           "defaultStorageQuota": {
+            "format": "int64",
             "minimum": 0,
-            "type": "number"
+            "type": "integer"
           },
           "enabled": {
             "type": "boolean"
@@ -13734,7 +13735,6 @@
           "buttonText",
           "clientId",
           "clientSecret",
-          "defaultStorageQuota",
           "enabled",
           "issuerUrl",
           "mobileOverrideEnabled",

open-api/immich-openapi-specs.json

@@ -1316,7 +1316,7 @@ export type SystemConfigOAuthDto = {
     buttonText: string;
     clientId: string;
     clientSecret: string;
-    defaultStorageQuota: number;
+    defaultStorageQuota?: number;
     enabled: boolean;
     issuerUrl: string;
     mobileOverrideEnabled: boolean;

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

@@ -89,7 +89,7 @@ export interface SystemConfig {
     buttonText: string;
     clientId: string;
     clientSecret: string;
-    defaultStorageQuota: number;
+    defaultStorageQuota?: number;
     enabled: boolean;
     issuerUrl: string;
     mobileOverrideEnabled: boolean;
@@ -253,7 +253,7 @@ export const defaults = Object.freeze<SystemConfig>({
     buttonText: 'Login with OAuth',
     clientId: '',
     clientSecret: '',
-    defaultStorageQuota: 0,
+    defaultStorageQuota: undefined,
     enabled: false,
     issuerUrl: '',
     mobileOverrideEnabled: false,

server/src/config.ts

@@ -360,7 +360,9 @@ class SystemConfigOAuthDto {
 
   @IsNumber()
   @Min(0)
-  defaultStorageQuota!: number;
+  @Optional()
+  @ApiProperty({ type: 'integer', format: 'int64' })
+  defaultStorageQuota?: number;
 
   @ValidateBoolean()
   enabled!: boolean;

server/src/dtos/system-config.dto.ts

@@ -702,7 +702,7 @@ describe(AuthService.name, () => {
       expect(mocks.user.create).toHaveBeenCalledWith(expect.objectContaining({ quotaSizeInBytes: 1_073_741_824 }));
     });
 
-    it('should not set quota for 0 quota', async () => {
+    it('should set quota for 0 quota', async () => {
       const user = factory.userAdmin({ oauthId: 'oauth-id' });
 
       mocks.systemMetadata.get.mockResolvedValue(systemConfigStub.oauthWithStorageQuota);
@@ -724,7 +724,7 @@ describe(AuthService.name, () => {
         email: user.email,
         name: ' ',
         oauthId: user.oauthId,
-        quotaSizeInBytes: null,
+        quotaSizeInBytes: 0,
         storageLabel: null,
       });
     });

server/src/services/auth.service.spec.ts

@@ -300,7 +300,7 @@ export class AuthService extends BaseService {
         name: userName,
         email: profile.email,
         oauthId: profile.sub,
-        quotaSizeInBytes: storageQuota * HumanReadableSize.GiB || null,
+        quotaSizeInBytes: storageQuota === undefined ? null : storageQuota * HumanReadableSize.GiB,
         storageLabel: storageLabel || null,
       });
     }

server/src/services/auth.service.ts

@@ -59,9 +59,6 @@ export const getKeysDeep = (target: unknown, path: string[] = []) => {
   const properties: string[] = [];
   for (const key of Object.keys(obj as object)) {
     const value = obj[key as keyof object];
-    if (value === undefined) {
-      continue;
-    }
 
     if (_.isObject(value) && !_.isArray(value) && !_.isDate(value)) {
       properties.push(...getKeysDeep(value, [...path, key]));

server/src/utils/misc.ts

@@ -182,7 +182,7 @@
                 label={$t('admin.oauth_storage_quota_default').toUpperCase()}
                 description={$t('admin.oauth_storage_quota_default_description')}
                 bind:value={config.oauth.defaultStorageQuota}
-                required={true}
+                required={false}
                 disabled={disabled || !config.oauth.enabled}
                 isEdited={!(config.oauth.defaultStorageQuota == savedConfig.oauth.defaultStorageQuota)}
               />