fix: remove the plus sign in front of the phone number (#5801)

logto-io · Apr 30, 2024 · 633a9a3 · 633a9a3
1 parent 24acae8
commit 633a9a3
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 3 deletions.
diff --git a/.changeset/smart-melons-shop.md b/.changeset/smart-melons-shop.md
@@ -0,0 +1,10 @@
+---
+"@logto/phrases": patch
+"@logto/core": patch
+---
+
+Fix file upload API.
+
+The `koa-body` has been upgraded to the latest version, which caused the file upload API to break. This change fixes the issue.
+
+The `ctx.request.files.file` in the new version is an array, so the code has been updated to pick the first one.
diff --git a/packages/connectors/connector-feishu-web/src/index.ts b/packages/connectors/connector-feishu-web/src/index.ts
@@ -153,7 +153,7 @@ export function getUserInfo(getConfig: GetConnectorConfig): GetUserInfo {
         avatar,
         email: conditional(email),
         userId: conditional(user_id),
-        phone: conditional(mobile),
+        phone: conditional(mobile?.replace('+', '')),
         rawData: jsonGuard.parse(response.body),
       };
     } catch (error: unknown) {

diff --git a/packages/core/src/routes/user-assets.ts b/packages/core/src/routes/user-assets.ts
@@ -52,13 +52,15 @@ export default function userAssetsRoutes<T extends ManagementApiRouter>(
     '/user-assets',
     koaGuard({
       files: object({
-        file: uploadFileGuard,
+        file: uploadFileGuard.array().min(1),
       }),
       response: userAssetsGuard,
     }),
     async (ctx, next) => {
-      const { file } = ctx.guard.files;
+      const { file: bodyFiles } = ctx.guard.files;
 
+      const file = bodyFiles[0];
+      assertThat(file, 'guard.invalid_input');
       assertThat(file.size <= maxUploadFileSize, 'guard.file_size_exceeded');
       assertThat(
         allowUploadMimeTypes.map(String).includes(file.mimetype),