fix: check of rate limit

ryo-ma · Sep 30, 2023 · 3a3622e · 3a3622e
1 parent 0728fac
commit 3a3622e
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 70 deletions.
diff --git a/.gitignore b/.gitignore
@@ -2,3 +2,4 @@
 .env
 .idea
 .lock
+*.sh
diff --git a/deno.lock b/deno.lock
diff --git a/src/Services/GithubApiService.ts b/src/Services/GithubApiService.ts
@@ -91,9 +91,11 @@ export class GithubApiService extends GithubRepository {
 
   private handleError(responseErrors: GithubError[]): ServiceError {
     const errors = responseErrors ?? [];
-    const isRateLimitExceeded = (errors ?? []).some((error) => {
-      error.type.includes(EServiceKindError.RATE_LIMIT);
-    });
+
+    const isRateLimitExceeded = errors.some((error) =>
+      error.type.includes(EServiceKindError.RATE_LIMIT) ||
+      error.message.includes("rate limit")
+    );
 
     if (isRateLimitExceeded) {
       throw new ServiceError(
@@ -133,6 +135,10 @@ export class GithubApiService extends GithubRepository {
       return response?.data?.data?.user ??
         new ServiceError("not found", EServiceKindError.NOT_FOUND);
     } catch (error) {
+      if (error instanceof ServiceError) {
+        Logger.error(error);
+        return error;
+      }
       // TODO: Move this to a logger instance later
       if (error instanceof Error && error.cause) {
         Logger.error(JSON.stringify(error.cause, null, 2));