refactor: remove dropbox, facebook & twitter login

Signed-off-by: Philip Molares <philip.molares@udo.edu>

backend/src/config/auth.config.ts

@@ -40,23 +40,10 @@ export interface AuthConfig {
     enableRegister: boolean;
     minimalPasswordStrength: number;
   };
-  facebook: {
-    clientID: string;
-    clientSecret: string;
-  };
-  twitter: {
-    consumerKey: string;
-    consumerSecret: string;
-  };
   github: {
     clientID: string;
     clientSecret: string;
   };
-  dropbox: {
-    clientID: string;
-    clientSecret: string;
-    appKey: string;
-  };
   google: {
     clientID: string;
     clientSecret: string;
@@ -134,29 +121,10 @@ const authSchema = Joi.object({
       .optional()
       .label('HD_AUTH_LOCAL_MINIMAL_PASSWORD_STRENGTH'),
   },
-  facebook: {
-    clientID: Joi.string().optional().label('HD_AUTH_FACEBOOK_CLIENT_ID'),
-    clientSecret: Joi.string()
-      .optional()
-      .label('HD_AUTH_FACEBOOK_CLIENT_SECRET'),
-  },
-  twitter: {
-    consumerKey: Joi.string().optional().label('HD_AUTH_TWITTER_CONSUMER_KEY'),
-    consumerSecret: Joi.string()
-      .optional()
-      .label('HD_AUTH_TWITTER_CONSUMER_SECRET'),
-  },
   github: {
     clientID: Joi.string().optional().label('HD_AUTH_GITHUB_CLIENT_ID'),
     clientSecret: Joi.string().optional().label('HD_AUTH_GITHUB_CLIENT_SECRET'),
   },
-  dropbox: {
-    clientID: Joi.string().optional().label('HD_AUTH_DROPBOX_CLIENT_ID'),
-    clientSecret: Joi.string()
-      .optional()
-      .label('HD_AUTH_DROPBOX_CLIENT_SECRET'),
-    appKey: Joi.string().optional().label('HD_AUTH_DROPBOX_APP_KEY'),
-  },
   google: {
     clientID: Joi.string().optional().label('HD_AUTH_GOOGLE_CLIENT_ID'),
     clientSecret: Joi.string().optional().label('HD_AUTH_GOOGLE_CLIENT_SECRET'),
@@ -379,23 +347,10 @@ export default registerAs('authConfig', () => {
           process.env.HD_AUTH_LOCAL_MINIMAL_PASSWORD_STRENGTH,
         ),
       },
-      facebook: {
-        clientID: process.env.HD_AUTH_FACEBOOK_CLIENT_ID,
-        clientSecret: process.env.HD_AUTH_FACEBOOK_CLIENT_SECRET,
-      },
-      twitter: {
-        consumerKey: process.env.HD_AUTH_TWITTER_CONSUMER_KEY,
-        consumerSecret: process.env.HD_AUTH_TWITTER_CONSUMER_SECRET,
-      },
       github: {
         clientID: process.env.HD_AUTH_GITHUB_CLIENT_ID,
         clientSecret: process.env.HD_AUTH_GITHUB_CLIENT_SECRET,
       },
-      dropbox: {
-        clientID: process.env.HD_AUTH_DROPBOX_CLIENT_ID,
-        clientSecret: process.env.HD_AUTH_DROPBOX_CLIENT_SECRET,
-        appKey: process.env.HD_AUTH_DROPBOX_APP_KEY,
-      },
       google: {
         clientID: process.env.HD_AUTH_GOOGLE_CLIENT_ID,
         clientSecret: process.env.HD_AUTH_GOOGLE_CLIENT_SECRET,

backend/src/config/mock/auth.config.mock.ts

@@ -1,5 +1,5 @@
 /*
- * SPDX-FileCopyrightText: 2022 The HedgeDoc developers (see AUTHORS file)
+ * SPDX-FileCopyrightText: 2023 The HedgeDoc developers (see AUTHORS file)
  *
  * SPDX-License-Identifier: AGPL-3.0-only
  */
@@ -19,23 +19,10 @@ export function createDefaultMockAuthConfig(): AuthConfig {
       enableRegister: true,
       minimalPasswordStrength: 2,
     },
-    facebook: {
-      clientID: '',
-      clientSecret: '',
-    },
-    twitter: {
-      consumerKey: '',
-      consumerSecret: '',
-    },
     github: {
       clientID: '',
       clientSecret: '',
     },
-    dropbox: {
-      clientID: '',
-      clientSecret: '',
-      appKey: '',
-    },
     google: {
       clientID: '',
       clientSecret: '',

backend/src/frontend-config/frontend-config.dto.ts

@@ -1,5 +1,5 @@
 /*
- * SPDX-FileCopyrightText: 2022 The HedgeDoc developers (see AUTHORS file)
+ * SPDX-FileCopyrightText: 2023 The HedgeDoc developers (see AUTHORS file)
  *
  * SPDX-License-Identifier: AGPL-3.0-only
  */
@@ -25,10 +25,7 @@ export enum AuthProviderType {
   SAML = 'saml',
   OAUTH2 = 'oauth2',
   GITLAB = 'gitlab',
-  FACEBOOK = 'facebook',
   GITHUB = 'github',
-  TWITTER = 'twitter',
-  DROPBOX = 'dropbox',
   GOOGLE = 'google',
 }
 
@@ -40,10 +37,7 @@ export type AuthProviderTypeWithCustomName =
 
 export type AuthProviderTypeWithoutCustomName =
   | AuthProviderType.LOCAL
-  | AuthProviderType.FACEBOOK
   | AuthProviderType.GITHUB
-  | AuthProviderType.TWITTER
-  | AuthProviderType.DROPBOX
   | AuthProviderType.GOOGLE;
 
 export class AuthProviderWithoutCustomNameDto extends BaseDto {

backend/src/frontend-config/frontend-config.service.spec.ts

@@ -1,5 +1,5 @@
 /*
- * SPDX-FileCopyrightText: 2022 The HedgeDoc developers (see AUTHORS file)
+ * SPDX-FileCopyrightText: 2023 The HedgeDoc developers (see AUTHORS file)
  *
  * SPDX-License-Identifier: AGPL-3.0-only
  */
@@ -37,23 +37,10 @@ describe('FrontendConfigService', () => {
       enableRegister: false,
       minimalPasswordStrength: 2,
     },
-    facebook: {
-      clientID: undefined,
-      clientSecret: undefined,
-    },
-    twitter: {
-      consumerKey: undefined,
-      consumerSecret: undefined,
-    },
     github: {
       clientID: undefined,
       clientSecret: undefined,
     },
-    dropbox: {
-      clientID: undefined,
-      clientSecret: undefined,
-      appKey: undefined,
-    },
     google: {
       clientID: undefined,
       clientSecret: undefined,
@@ -66,23 +53,10 @@ describe('FrontendConfigService', () => {
   };
 
   describe('getAuthProviders', () => {
-    const facebook: AuthConfig['facebook'] = {
-      clientID: 'facebookTestId',
-      clientSecret: 'facebookTestSecret',
-    };
-    const twitter: AuthConfig['twitter'] = {
-      consumerKey: 'twitterTestId',
-      consumerSecret: 'twitterTestSecret',
-    };
     const github: AuthConfig['github'] = {
       clientID: 'githubTestId',
       clientSecret: 'githubTestSecret',
     };
-    const dropbox: AuthConfig['dropbox'] = {
-      clientID: 'dropboxTestId',
-      clientSecret: 'dropboxTestSecret',
-      appKey: 'dropboxTestKey',
-    };
     const google: AuthConfig['google'] = {
       clientID: 'googleTestId',
       clientSecret: 'googleTestSecret',
@@ -155,10 +129,7 @@ describe('FrontendConfigService', () => {
       },
     ];
     for (const authConfigConfigured of [
-      facebook,
-      twitter,
       github,
-      dropbox,
       google,
       gitlab,
       ldap,
@@ -211,16 +182,6 @@ describe('FrontendConfigService', () => {
         }).compile();
         const service = module.get(FrontendConfigService);
         const config = await service.getFrontendConfig();
-        if (authConfig.dropbox.clientID) {
-          expect(config.authProviders).toContainEqual({
-            type: AuthProviderType.DROPBOX,
-          });
-        }
-        if (authConfig.facebook.clientID) {
-          expect(config.authProviders).toContainEqual({
-            type: AuthProviderType.FACEBOOK,
-          });
-        }
         if (authConfig.google.clientID) {
           expect(config.authProviders).toContainEqual({
             type: AuthProviderType.GOOGLE,
@@ -236,11 +197,6 @@ describe('FrontendConfigService', () => {
             type: AuthProviderType.LOCAL,
           });
         }
-        if (authConfig.twitter.consumerKey) {
-          expect(config.authProviders).toContainEqual({
-            type: AuthProviderType.TWITTER,
-          });
-        }
         expect(
           config.authProviders.filter(
             (provider) => provider.type === AuthProviderType.GITLAB,

backend/src/frontend-config/frontend-config.service.ts

@@ -1,5 +1,5 @@
 /*
- * SPDX-FileCopyrightText: 2022 The HedgeDoc developers (see AUTHORS file)
+ * SPDX-FileCopyrightText: 2023 The HedgeDoc developers (see AUTHORS file)
  *
  * SPDX-License-Identifier: AGPL-3.0-only
  */
@@ -66,16 +66,6 @@ export class FrontendConfigService {
         type: AuthProviderType.LOCAL,
       });
     }
-    if (this.authConfig.dropbox.clientID) {
-      providers.push({
-        type: AuthProviderType.DROPBOX,
-      });
-    }
-    if (this.authConfig.facebook.clientID) {
-      providers.push({
-        type: AuthProviderType.FACEBOOK,
-      });
-    }
     if (this.authConfig.github.clientID) {
       providers.push({
         type: AuthProviderType.GITHUB,
@@ -86,11 +76,6 @@ export class FrontendConfigService {
         type: AuthProviderType.GOOGLE,
       });
     }
-    if (this.authConfig.twitter.consumerKey) {
-      providers.push({
-        type: AuthProviderType.TWITTER,
-      });
-    }
     this.authConfig.gitlab.forEach((gitLabEntry) => {
       providers.push({
         type: AuthProviderType.GITLAB,

backend/src/identity/identity.entity.ts

@@ -1,5 +1,5 @@
 /*
- * SPDX-FileCopyrightText: 2021 The HedgeDoc developers (see AUTHORS file)
+ * SPDX-FileCopyrightText: 2023 The HedgeDoc developers (see AUTHORS file)
  *
  * SPDX-License-Identifier: AGPL-3.0-only
  */
@@ -18,7 +18,7 @@ import { ProviderType } from './provider-type.enum';
 /**
  * The identity represents a single way for a user to login.
  * A 'user' can have any number of these.
- * Each one holds a type (local, github, twitter, etc.), if this type can have multiple instances (e.g. gitlab),
+ * Each one holds a type (local, github, etc.), if this type can have multiple instances (e.g. gitlab),
  * it also saves the name of the instance. Also if this identity shall be the syncSource is saved.
  */
 @Entity()

backend/src/identity/provider-type.enum.ts

@@ -1,5 +1,5 @@
 /*
- * SPDX-FileCopyrightText: 2021 The HedgeDoc developers (see AUTHORS file)
+ * SPDX-FileCopyrightText: 2023 The HedgeDoc developers (see AUTHORS file)
  *
  * SPDX-License-Identifier: AGPL-3.0-only
  */
@@ -11,8 +11,5 @@ export enum ProviderType {
   OAUTH2 = 'oauth2',
   GITLAB = 'gitlab',
   GITHUB = 'github',
-  FACEBOOK = 'facebook',
-  TWITTER = 'twitter',
-  DROPBOX = 'dropbox',
   GOOGLE = 'google',
 }