refactor: remove web folder

a60814billy · a60814billy · commit b4ec353fcd5c · 2020-01-06T14:19:02.000+08:00
Signed-off-by: BoHong Li &lt;raccoon@hackmd.io&gt;
diff --git a/app.js b/app.js
@@ -153,7 +153,7 @@ server.on('resumeSession', function (id, cb) {
 })
 
 // middleware which blocks requests when we're too busy
-app.use(require('./lib/web/middleware/tooBusy'))
+app.use(require('./lib/middleware/tooBusy'))
 
 app.use(flash())
 
@@ -162,10 +162,10 @@ app.use(passport.initialize())
 app.use(passport.session())
 
 // check uri is valid before going further
-app.use(require('./lib/web/middleware/checkURIValid'))
+app.use(require('./lib/middleware/checkURIValid'))
 // redirect url without trailing slashes
-app.use(require('./lib/web/middleware/redirectWithoutTrailingSlashes'))
-app.use(require('./lib/web/middleware/codiMDVersion'))
+app.use(require('./lib/middleware/redirectWithoutTrailingSlashes'))
+app.use(require('./lib/middleware/codiMDVersion'))
 
 // routes need sessions
 // template files
@@ -206,7 +206,7 @@ app.locals.enableDropBoxSave = config.isDropboxEnable
 app.locals.enableGitHubGist = config.isGitHubEnable
 app.locals.enableGitlabSnippets = config.isGitlabSnippetsEnable
 
-app.use(require('./lib/web/routes').router)
+app.use(require('./lib/routes').router)
 
 // response not found if no any route matxches
 app.get('*', function (req, res) {
diff --git a/lib/auth/bitbucket/index.js b/lib/auth/bitbucket/index.js
@@ -3,7 +3,7 @@
 const Router = require('express').Router
 const passport = require('passport')
 const BitbucketStrategy = require('passport-bitbucket-oauth2').Strategy
-const config = require('../../../config')
+const config = require('../../config')
 const { setReturnToFromReferer, passportGeneralCallback } = require('../utils')
 
 const bitbucketAuth = module.exports = Router()
diff --git a/lib/auth/dropbox/index.js b/lib/auth/dropbox/index.js
@@ -3,7 +3,7 @@
 const Router = require('express').Router
 const passport = require('passport')
 const DropboxStrategy = require('passport-dropbox-oauth2').Strategy
-const config = require('../../../config')
+const config = require('../../config')
 const { setReturnToFromReferer, passportGeneralCallback } = require('../utils')
 
 const dropboxAuth = module.exports = Router()
diff --git a/lib/auth/email/index.js b/lib/auth/email/index.js
@@ -4,12 +4,12 @@ const Router = require('express').Router
 const passport = require('passport')
 const validator = require('validator')
 const LocalStrategy = require('passport-local').Strategy
-const config = require('../../../config')
-const models = require('../../../models')
-const logger = require('../../../logger')
+const config = require('../../config')
+const models = require('../../models')
+const logger = require('../../logger')
 const { setReturnToFromReferer } = require('../utils')
 const { urlencodedParser } = require('../../utils')
-const response = require('../../../response')
+const response = require('../../response')
 
 const emailAuth = module.exports = Router()
 
diff --git a/lib/auth/facebook/index.js b/lib/auth/facebook/index.js
@@ -4,7 +4,7 @@ const Router = require('express').Router
 const passport = require('passport')
 const FacebookStrategy = require('passport-facebook').Strategy
 
-const config = require('../../../config')
+const config = require('../../config')
 const { setReturnToFromReferer, passportGeneralCallback } = require('../utils')
 
 const facebookAuth = module.exports = Router()
diff --git a/lib/auth/github/index.js b/lib/auth/github/index.js
@@ -3,8 +3,8 @@
 const Router = require('express').Router
 const passport = require('passport')
 const GithubStrategy = require('passport-github').Strategy
-const config = require('../../../config')
-const response = require('../../../response')
+const config = require('../../config')
+const response = require('../../response')
 const { setReturnToFromReferer, passportGeneralCallback } = require('../utils')
 const { URL } = require('url')
 
diff --git a/lib/auth/gitlab/index.js b/lib/auth/gitlab/index.js
@@ -3,8 +3,8 @@
 const Router = require('express').Router
 const passport = require('passport')
 const GitlabStrategy = require('passport-gitlab2').Strategy
-const config = require('../../../config')
-const response = require('../../../response')
+const config = require('../../config')
+const response = require('../../response')
 const { setReturnToFromReferer, passportGeneralCallback } = require('../utils')
 const HttpsProxyAgent = require('https-proxy-agent')
 
diff --git a/lib/auth/google/index.js b/lib/auth/google/index.js
@@ -3,7 +3,7 @@
 const Router = require('express').Router
 const passport = require('passport')
 var GoogleStrategy = require('passport-google-oauth20').Strategy
-const config = require('../../../config')
+const config = require('../../config')
 const { setReturnToFromReferer, passportGeneralCallback } = require('../utils')
 
 const googleAuth = module.exports = Router()
diff --git a/lib/auth/index.js b/lib/auth/index.js
@@ -3,9 +3,9 @@
 const Router = require('express').Router
 const passport = require('passport')
 
-const config = require('../../config')
-const logger = require('../../logger')
-const models = require('../../models')
+const config = require('../config')
+const logger = require('../logger')
+const models = require('../models')
 
 const authRouter = module.exports = Router()
 
diff --git a/lib/auth/ldap/index.js b/lib/auth/ldap/index.js
@@ -3,12 +3,12 @@
 const Router = require('express').Router
 const passport = require('passport')
 const LDAPStrategy = require('passport-ldapauth')
-const config = require('../../../config')
-const models = require('../../../models')
-const logger = require('../../../logger')
+const config = require('../../config')
+const models = require('../../models')
+const logger = require('../../logger')
 const { setReturnToFromReferer } = require('../utils')
 const { urlencodedParser } = require('../../utils')
-const response = require('../../../response')
+const response = require('../../response')
 
 const ldapAuth = module.exports = Router()
 
diff --git a/lib/auth/mattermost/index.js b/lib/auth/mattermost/index.js
@@ -5,7 +5,7 @@ const Router = require('express').Router
 const passport = require('passport')
 const MattermostClient = require('mattermost-redux/client/client4').default
 const OAuthStrategy = require('passport-oauth2').Strategy
-const config = require('../../../config')
+const config = require('../../config')
 const { setReturnToFromReferer, passportGeneralCallback } = require('../utils')
 
 const mattermostAuth = module.exports = Router()
diff --git a/lib/auth/oauth2/index.js b/lib/auth/oauth2/index.js
@@ -3,7 +3,7 @@
 const Router = require('express').Router
 const passport = require('passport')
 const { Strategy, InternalOAuthError } = require('passport-oauth2')
-const config = require('../../../config')
+const config = require('../../config')
 const { setReturnToFromReferer, passportGeneralCallback } = require('../utils')
 
 const oauth2Auth = module.exports = Router()
diff --git a/lib/auth/openid/index.js b/lib/auth/openid/index.js
@@ -3,9 +3,9 @@
 const Router = require('express').Router
 const passport = require('passport')
 const OpenIDStrategy = require('@passport-next/passport-openid').Strategy
-const config = require('../../../config')
-const models = require('../../../models')
-const logger = require('../../../logger')
+const config = require('../../config')
+const models = require('../../models')
+const logger = require('../../logger')
 const { urlencodedParser } = require('../../utils')
 const { setReturnToFromReferer } = require('../utils')
 
diff --git a/lib/auth/saml/index.js b/lib/auth/saml/index.js
@@ -3,9 +3,9 @@
 const Router = require('express').Router
 const passport = require('passport')
 const SamlStrategy = require('passport-saml').Strategy
-const config = require('../../../config')
-const models = require('../../../models')
-const logger = require('../../../logger')
+const config = require('../../config')
+const models = require('../../models')
+const logger = require('../../logger')
 const { urlencodedParser } = require('../../utils')
 const fs = require('fs')
 const intersection = function (array1, array2) { return array1.filter((n) => array2.includes(n)) }
diff --git a/lib/auth/twitter/index.js b/lib/auth/twitter/index.js
@@ -4,7 +4,7 @@ const Router = require('express').Router
 const passport = require('passport')
 const TwitterStrategy = require('passport-twitter').Strategy
 
-const config = require('../../../config')
+const config = require('../../config')
 const { setReturnToFromReferer, passportGeneralCallback } = require('../utils')
 
 const twitterAuth = module.exports = Router()
diff --git a/lib/auth/utils.js b/lib/auth/utils.js
@@ -1,8 +1,8 @@
 'use strict'
 
-const models = require('../../models')
-const config = require('../../config')
-const logger = require('../../logger')
+const models = require('../models')
+const config = require('../config')
+const logger = require('../logger')
 
 exports.setReturnToFromReferer = function setReturnToFromReferer (req) {
   var referer = req.get('referer')
diff --git a/lib/errorPage/index.js b/lib/errorPage/index.js
@@ -1,7 +1,7 @@
 'use strict'
 
-const config = require('../../config')
-const { responseError } = require('../../response')
+const config = require('../config')
+const { responseError } = require('../response')
 
 exports.errorForbidden = (req, res) => {
   if (req.user) {
diff --git a/lib/history/index.js b/lib/history/index.js
@@ -4,10 +4,10 @@
 var LZString = require('@hackmd/lz-string')
 
 // core
-var config = require('../../config')
-var logger = require('../../logger')
-var response = require('../../response')
-var models = require('../../models')
+var config = require('../config')
+var logger = require('../logger')
+var response = require('../response')
+var models = require('../models')
 
 function getHistory (userid, callback) {
   models.User.findOne({
diff --git a/lib/homepage/index.js b/lib/homepage/index.js
@@ -2,9 +2,9 @@
 
 const fs = require('fs')
 const path = require('path')
-const config = require('../../config')
-const { User } = require('../../models')
-const logger = require('../../logger')
+const config = require('../config')
+const { User } = require('../models')
+const logger = require('../logger')
 
 exports.showIndex = async (req, res) => {
   const isLogin = req.isAuthenticated()
diff --git a/lib/imageRouter/azure.js b/lib/imageRouter/azure.js
@@ -1,8 +1,8 @@
 'use strict'
 const path = require('path')
 
-const config = require('../../config')
-const logger = require('../../logger')
+const config = require('../config')
+const logger = require('../logger')
 
 const azure = require('azure-storage')
 
diff --git a/lib/imageRouter/filesystem.js b/lib/imageRouter/filesystem.js
@@ -2,8 +2,8 @@
 const URL = require('url').URL
 const path = require('path')
 
-const config = require('../../config')
-const logger = require('../../logger')
+const config = require('../config')
+const logger = require('../logger')
 
 exports.uploadImage = function (imagePath, callback) {
   if (!imagePath || typeof imagePath !== 'string') {
diff --git a/lib/imageRouter/imgur.js b/lib/imageRouter/imgur.js
@@ -1,6 +1,6 @@
 'use strict'
-const config = require('../../config')
-const logger = require('../../logger')
+const config = require('../config')
+const logger = require('../logger')
 
 const imgur = require('@hackmd/imgur')
 
diff --git a/lib/imageRouter/index.js b/lib/imageRouter/index.js
@@ -3,9 +3,9 @@
 const Router = require('express').Router
 const formidable = require('formidable')
 
-const config = require('../../config')
-const logger = require('../../logger')
-const response = require('../../response')
+const config = require('../config')
+const logger = require('../logger')
+const response = require('../response')
 
 const imageRouter = module.exports = Router()
 
diff --git a/lib/imageRouter/lutim.js b/lib/imageRouter/lutim.js
@@ -1,8 +1,8 @@
 'use strict'
-const config = require('../../config')
-const logger = require('../../logger')
+const config = require('../config')
+const logger = require('../logger')
 
-const lutim = require('lutim')
+const lutim = require('lib/imageRouter/lutim')
 
 exports.uploadImage = function (imagePath, callback) {
   if (!imagePath || typeof imagePath !== 'string') {
diff --git a/lib/imageRouter/minio.js b/lib/imageRouter/minio.js
@@ -2,11 +2,11 @@
 const fs = require('fs')
 const path = require('path')
 
-const config = require('../../config')
-const { getImageMimeType } = require('../../utils')
-const logger = require('../../logger')
+const config = require('../config')
+const { getImageMimeType } = require('../utils')
+const logger = require('../logger')
 
-const Minio = require('minio')
+const Minio = require('lib/imageRouter/minio')
 const minioClient = new Minio.Client({
   endPoint: config.minio.endPoint,
   port: config.minio.port,
diff --git a/lib/imageRouter/s3.js b/lib/imageRouter/s3.js
@@ -2,9 +2,9 @@
 const fs = require('fs')
 const path = require('path')
 
-const config = require('../../config')
-const { getImageMimeType } = require('../../utils')
-const logger = require('../../logger')
+const config = require('../config')
+const { getImageMimeType } = require('../utils')
+const logger = require('../logger')
 
 const AWS = require('aws-sdk')
 const awsConfig = new AWS.Config(config.s3)
diff --git a/lib/middleware/checkURIValid.js b/lib/middleware/checkURIValid.js
@@ -1,7 +1,7 @@
 'use strict'
 
-const logger = require('../../logger')
-const response = require('../../response')
+const logger = require('../logger')
+const response = require('../response')
 
 module.exports = function (req, res, next) {
   try {
diff --git a/lib/middleware/codiMDVersion.js b/lib/middleware/codiMDVersion.js
@@ -1,6 +1,6 @@
 'use strict'
 
-const config = require('../../config')
+const config = require('../config')
 
 module.exports = function (req, res, next) {
   res.set({
diff --git a/lib/middleware/redirectWithoutTrailingSlashes.js b/lib/middleware/redirectWithoutTrailingSlashes.js
@@ -1,6 +1,6 @@
 'use strict'
 
-const config = require('../../config')
+const config = require('../config')
 
 module.exports = function (req, res, next) {
   if (req.method === 'GET' && req.path.substr(-1) === '/' && req.path.length > 1) {
diff --git a/lib/middleware/tooBusy.js b/lib/middleware/tooBusy.js
@@ -2,8 +2,8 @@
 
 const toobusy = require('toobusy-js')
 
-const config = require('../../config')
-const response = require('../../response')
+const config = require('../config')
+const response = require('../response')
 
 toobusy.maxLag(config.responseMaxLag)
 
diff --git a/lib/realtime/realtime.js b/lib/realtime/realtime.js
@@ -14,7 +14,7 @@ const get = require('lodash/get')
 // core
 const config = require('../config')
 const logger = require('../logger')
-const history = require('../web/history')
+const history = require('../history')
 const models = require('../models')
 
 // ot
diff --git a/lib/response.js b/lib/response.js
@@ -14,7 +14,7 @@ const config = require('./config')
 const logger = require('./logger')
 const models = require('./models')
 const utils = require('./utils')
-const history = require('./web/history')
+const history = require('./history')
 
 // public
 exports.responseError = responseError
diff --git a/lib/routes.js b/lib/routes.js
@@ -10,7 +10,7 @@ const errorPageController = require('./errorPage')
 const statusController = require('./status')
 const historyController = require('./history')
 const userController = require('./user')
-const response = require('../response')
+const response = require('./response')
 const appRouter = Router()
 
 // register route
diff --git a/lib/status/index.js b/lib/status/index.js
@@ -1,7 +1,7 @@
 'use strict'
 
-const realtime = require('../../realtime/realtime')
-const config = require('../../config')
+const realtime = require('../realtime/realtime')
+const config = require('../config')
 
 exports.getStatus = async (req, res) => {
   const data = await realtime.getStatus()
diff --git a/lib/user/index.js b/lib/user/index.js
diff --git a/lib/utils.js b/lib/utils.js
diff --git a/lib/web/utils.js b/lib/web/utils.js
