fix: Fixed error if you dont have access to project

BottlecapDave · Jul 8, 2023 · 88059d1 · 88059d1
1 parent 3834bef
commit 88059d1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/reminder/Dockerfile b/reminder/Dockerfile
@@ -1,4 +1,4 @@
-FROM node:18.16-stretch-slim as build
+FROM node:18.16-buster-slim as build
 
 WORKDIR /app
 
@@ -10,7 +10,7 @@ COPY src/. src/.
 
 RUN npm run build
 
-FROM node:16.13-stretch-slim as deploy
+FROM node:18.16-buster-slim as deploy
 WORKDIR /app
 
 COPY --from=build /app/lib /app/lib

diff --git a/reminder/src/gitlabService.ts b/reminder/src/gitlabService.ts
@@ -43,7 +43,7 @@ export class GitlabService {
 
             const project = projectResp.data;
             const resp = await axios.get(`${request.gitlabBaseURL}/api/v4/projects/${projectId}/merge_requests?state=opened&scope=all&sort=asc`, requestConfig);
-            if (projectResp.status === 403) {
+            if (resp.status === 403) {
                 throw new Error(`Not authorised to see merge requests for project '${projectId}'`);
             }