Merge pull request #223 from learnsoftwaredevelopment/resolved-video-host-whitelist-bug

Resolved a bug where previously video host with subdomain `www.` was not part of whitelist

Author: JonathanLeeWH

models/software.js

@@ -1,6 +1,7 @@
 const mongoose = require('mongoose');
 const { isURL } = require('validator');
 const uniqueValidator = require('mongoose-unique-validator');
+const { ALLOWED_VIDEO_HOST_WHITELIST } = require('../utils/config');
 
 const softwareSchema = new mongoose.Schema(
   {
@@ -101,7 +102,7 @@ const softwareSchema = new mongoose.Schema(
           if (value) {
             return isURL(value, {
               protocols: ['http', 'https'],
-              host_whitelist: ['youtube.com', 'vimeo.com'],
+              host_whitelist: ALLOWED_VIDEO_HOST_WHITELIST,
             });
           }
           return true;

requests/api/auth/post_auth_login.rest

@@ -2,6 +2,6 @@ POST http://localhost:3001/api/auth/login
 Content-Type: application/json
 
 {
-    "email": "sample@example.com",
-    "password": "SamplePassword"
+    "email": "lecamiv920@activesniper.com",
+    "password": "password"
 }

utils/config.js

@@ -11,11 +11,18 @@ const MONGODB_URI = NODE_ENVIRONMENT === 'test'
 
 const { FIREBASE_CLIENT_API_KEY } = process.env;
 const ALLOWED_USERNAME_REGEX = '^[a-z0-9_]+$';
+const ALLOWED_VIDEO_HOST_WHITELIST = [
+  'youtube.com',
+  'vimeo.com',
+  'www.youtube.com',
+  'www.vimeo.com',
+];
 
 module.exports = {
   PORT,
   NODE_ENVIRONMENT,
   MONGODB_URI,
   ALLOWED_USERNAME_REGEX,
   FIREBASE_CLIENT_API_KEY,
+  ALLOWED_VIDEO_HOST_WHITELIST
 };