log installation failures (#372)

* log installation failures

* increase version

* fix

* fix log

* typo

Author: gunzip

package.json

@@ -1,6 +1,6 @@
 {
   "name": "italia-backend",
-  "version": "0.3.3",
+  "version": "0.3.4",
   "description": "Italia app and web backend",
   "main": "index.js",
   "engines": {
@@ -137,4 +137,4 @@
       "!src/**/*.d.ts"
     ]
   }
-}
+}

src/controllers/notificationController.ts

@@ -45,8 +45,11 @@ export default class NotificationController {
     const errorOrUser = extractUserFromRequest(req);
 
     if (isLeft(errorOrUser)) {
-      // Unable to extract the user from the request.
       const error = errorOrUser.value;
+      log.error(
+        "Unable to extract the user from the request: %s",
+        error.message
+      );
       return ResponseErrorInternal(error.message);
     }
 

src/services/notificationService.ts

@@ -22,6 +22,7 @@ import {
   INotificationTemplate,
   toFiscalCodeHash
 } from "../types/notification";
+import { log } from "../utils/logger";
 
 /**
  * A template suitable for Apple's APNs.
@@ -105,6 +106,13 @@ export default class NotificationService {
         // tslint:disable-next-line:no-any
         (azureInstallation as any) as string,
         (error, response) => {
+          if (error) {
+            log.error(
+              "Unable to create installation: %s (error=%s)",
+              JSON.stringify(azureInstallation),
+              error.message
+            );
+          }
           return resolve(this.buildResponse(error, response));
         }
       );