Skip to content
This repository has been archived by the owner on Jan 1, 2024. It is now read-only.

Commit

Permalink
fix(server): use language middleware instead of user middleware
Browse files Browse the repository at this point in the history
fix(server): use language middleware instead of user middleware
  • Loading branch information
Metnew committed Feb 18, 2018
1 parent cff2618 commit 554bef0
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 71 deletions.
52 changes: 0 additions & 52 deletions src/server/middlewares/auth.js

This file was deleted.

22 changes: 3 additions & 19 deletions src/server/middlewares/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,9 @@ import cookieParser from 'cookie-parser'
import bodyParser from 'body-parser'
import morgan from 'morgan'
import Raven from 'raven'
import authMiddleware from './auth'
import languageMiddleware from './language'

export default (app: express$Application): express$Application => {
// Must configure Raven before doing anything else with it
Raven.config(process.env.SENTRY_DSN).install()
// The request handler must be the first middleware on the app
app.use(Raven.requestHandler())
Expand All @@ -24,29 +23,14 @@ export default (app: express$Application): express$Application => {
app.use(morgan('dev'))
app.use(cookieParser())
app.use(
express.static(process.env.CLIENT_DIST_PATH, {
express.static(process.env.CLIENT_STATIC_PATH, {
index: false
})
)
app.use(bodyParser.json())
app.use(authMiddleware)
app.use(languageMiddleware)
// The error handler must be before any other error middleware
app.use(Raven.errorHandler())
// Optional fallthrough error handler
// eslint-disable-next-line
app.use(function onError(
err,
req: express$Request,
res: express$Response,
next: express$NextFunction
) {
// NOTE: @Metnew: line disabled by ESlint, because err is already handled by sentry
//
// The error id is attached to `res.sentry` to be returned
// and optionally displayed to the user for support.
res.statusCode = 500
res.end(res.sentry + '\n')
})

return app
}
11 changes: 11 additions & 0 deletions src/server/middlewares/language.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
// @flow
import {supportedLanguages, defaultLanguage} from '../i18n'

export default (req, res, next) => {
const language: string =
req.acceptsLanguages(supportedLanguages) || defaultLanguage
req.user = {
language
}
next()
}

0 comments on commit 554bef0

Please sign in to comment.