فهرست منبع

improved err handling

kaori 5 سال پیش
والد
کامیت
25e137d9d4
2فایلهای تغییر یافته به همراه10 افزوده شده و 4 حذف شده
  1. 5 1
      packages/slackbot-proxy/src/controllers/slack.ts
  2. 5 3
      packages/slackbot-proxy/src/services/RegisterService.ts

+ 5 - 1
packages/slackbot-proxy/src/controllers/slack.ts

@@ -187,7 +187,11 @@ export class SlackCtrl {
 
     // register
     if (callBackId === 'register') {
-      await this.registerService.insertOrderRecord(this.orderRepository, installation, authorizeResult, payload);
+      const insertOrderResults = await this.registerService.insertOrderRecord(this.orderRepository, installation, authorizeResult, payload);
+      if (insertOrderResults != null && insertOrderResults.error != null) {
+        return;
+      }
+
       await this.registerService.notifyServerUriToSlack(authorizeResult, payload);
       return;
     }

+ 5 - 3
packages/slackbot-proxy/src/services/RegisterService.ts

@@ -73,12 +73,14 @@ export class RegisterService implements GrowiCommandProcessor {
       });
     };
 
+    let url;
+
     try {
-      const url = new URL(growiUrl);
-      return url;
+      url = new URL(growiUrl);
     }
-    catch (err) {
+    catch (error) {
       postInvalidUrlErr();
+      return { error, growiUrl: url };
     }
 
     orderRepository.save({