♻️ Make gravatar optional (fixed #1114)

staart · Jan 9, 2021 · b3418d0 · b3418d0
1 parent 243857f
commit b3418d0
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 10 deletions.
diff --git a/src/config/configuration.interface.ts b/src/config/configuration.interface.ts
@@ -117,4 +117,8 @@ export interface Configuration {
   googleMaps: {
     apiKey: string;
   };
+
+  gravatar: {
+    enabled: boolean;
+  };
 }
diff --git a/src/config/configuration.ts b/src/config/configuration.ts
@@ -140,6 +140,9 @@ const configuration: Configuration = {
   googleMaps: {
     apiKey: process.env.GOOGLE_MAPS_API_KEY,
   },
+  gravatar: {
+    enabled: bool(process.env.PASSWORD_PWNED_CHECK, true),
+  },
 };
 
 const configFunction: ConfigFactory<Configuration> = () => configuration;

diff --git a/src/modules/auth/auth.service.ts b/src/modules/auth/auth.service.ts
@@ -201,16 +201,18 @@ export class AuthService {
       } catch (error) {}
     }
 
-    for await (const emailString of [email, emailSafe]) {
-      const md5Email = createHash('md5').update(emailString).digest('hex');
-      try {
-        const img = await got(
-          `https://www.gravatar.com/avatar/${md5Email}?d=404`,
-          { responseType: 'buffer' },
-        );
-        if (img.body.byteLength > 1)
-          data.profilePictureUrl = `https://www.gravatar.com/avatar/${md5Email}?d=mp`;
-      } catch (error) {}
+    if (this.configService.get<boolean>('gravatar.enabled')) {
+      for await (const emailString of [email, emailSafe]) {
+        const md5Email = createHash('md5').update(emailString).digest('hex');
+        try {
+          const img = await got(
+            `https://www.gravatar.com/avatar/${md5Email}?d=404`,
+            { responseType: 'buffer' },
+          );
+          if (img.body.byteLength > 1)
+            data.profilePictureUrl = `https://www.gravatar.com/avatar/${md5Email}?d=mp`;
+        } catch (error) {}
+      }
     }
 
     let id: number | undefined = undefined;