From 442e11c419cfab56c3145806e06e44ba2f7a24e1 Mon Sep 17 00:00:00 2001
From: Pavel Gnedov <pavel@gnedov.info>
Date: Fri, 27 May 2022 11:24:31 +0700
Subject: [PATCH] =?UTF-8?q?=D0=94=D0=BE=D0=B1=D0=B0=D0=B2=D0=BB=D0=B5?=
 =?UTF-8?q?=D0=BD=20=D0=B2=D1=8B=D0=B2=D0=BE=D0=B4=20debug-=D0=BB=D0=BE?=
 =?UTF-8?q?=D0=B3=D0=BE=D0=B2?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../redmine-data-loader/redmine-data-loader.ts    | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)

diff --git a/libs/event-emitter/src/redmine-data-loader/redmine-data-loader.ts b/libs/event-emitter/src/redmine-data-loader/redmine-data-loader.ts
index 6df42ee..861baa6 100644
--- a/libs/event-emitter/src/redmine-data-loader/redmine-data-loader.ts
+++ b/libs/event-emitter/src/redmine-data-loader/redmine-data-loader.ts
@@ -1,28 +1,35 @@
 import axios from 'axios';
 import { RedmineIssueData } from '../models/RedmineIssueData';
-import { Injectable } from '@nestjs/common';
+import { Injectable, Logger } from '@nestjs/common';
 import { ConfigService } from '@nestjs/config';
 
 @Injectable()
 export class RedmineDataLoader {
   urlPrefix: string;
 
+  private logger = new Logger(RedmineDataLoader.name);
+
   constructor(private configService: ConfigService) {
     this.urlPrefix = this.configService.get<string>('redmineUrlPrefix');
   }
 
-  async loadIssues(issues: number[]): Promise<RedmineIssueData[]> {
+  async loadIssues(issues: number[]): Promise<(RedmineIssueData | null)[]> {
     const promises = issues.map((issue) => this.loadIssue(issue));
     return Promise.all(promises);
   }
 
-  async loadIssue(issueNumber: number): Promise<RedmineIssueData> {
+  async loadIssue(issueNumber: number): Promise<RedmineIssueData | null> {
     const url = this.getUrl(issueNumber);
     const resp = await axios.get(url);
     if (!resp || !resp.data || !resp.data.issue) {
-      console.error('Failed to load data from redmine');
+      this.logger.error(
+        `Failed to load data for from redmine, issueNumber = ${issueNumber}`,
+      );
       return null;
     }
+    this.logger.debug(
+      `Loaded issue, issueNumber = ${issueNumber}, subject = ${resp.data.issue.subject}`,
+    );
     return resp.data.issue;
   }