Skip to content

Commit a112d09

Browse files
AnnineAnnine
authored andcommitted
cleaning code
1 parent 05ab21c commit a112d09

File tree

4 files changed

+47
-24
lines changed

4 files changed

+47
-24
lines changed

src/config/googleToken.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import GoogleTokenStrategy from 'passport-google-id-token'
22
import { CLIENT_ID } from '../util/secrets'
3-
import User from 'src/models/User'
4-
import { nextTick } from 'async'
3+
import User from '../models/User'
54

65
export const passportGoogleIdToken = () =>
76
new GoogleTokenStrategy(
@@ -10,7 +9,7 @@ export const passportGoogleIdToken = () =>
109
},
1110
async function (parsedToken: any, googleId: any, done: any) {
1211
//Create newUser Account here
13-
const user = await User.findEmailandCreate(parsedToken)
12+
const user = await User.findEmailandCreate(parsedToken.payload)
1413
done(null, user)
1514
}
1615
)

src/middlewares/jwt.ts

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
import jwt from 'jsonwebtoken'
2+
import { SESSION_SECRET } from '../util/secrets'
3+
import user from 'src/services/user'
4+
5+
export type Data = {
6+
name: string;
7+
email: string;
8+
isAdmin: boolean;
9+
}
10+
11+
const data: Data = { name: user.name, email: user.email, isAdmin: user.isAdmin }
12+
13+
const createJWT = jwt.sign(data, SESSION_SECRET, { expiresIn: '1h' })
14+
15+
const verifyJWT = jwt.verify(createJWT, SESSION_SECRET)
16+
17+
const checkAdmin: boolean = verifyJWT.isAdmin
18+
19+
export default {
20+
createJWT,
21+
verifyJWT,
22+
checkAdmin,
23+
}

src/middlewares/validator.ts

Lines changed: 20 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1,22 +1,22 @@
1-
import { Request, Response, NextFunction } from 'express'
2-
import { body, validationResult } from 'express-validator'
1+
// import { Request, Response, NextFunction } from 'express'
2+
// import { body, validationResult } from 'express-validator'
33

4-
export const userValidationRules = () => {
5-
return [
6-
// must have an email
7-
body('email').isEmail(),
8-
//pass at least 8 chars long
9-
body('password').isLength({ min: 8 }),
10-
]
11-
}
4+
// export const userValidationRules = () => {
5+
// return [
6+
// // must have an email
7+
// body('email').isEmail(),
8+
// //pass at least 8 chars long
9+
// body('password').isLength({ min: 8 }),
10+
// ]
11+
// }
1212

13-
export const validate = (validations: any) => {
14-
return async (req: Request, res: Response, next: NextFunction) => {
15-
await Promise.all(validations.map((validation: any) => validation.run(req)))
16-
const errors = validationResult(req)
17-
if (errors.isEmpty()) {
18-
return next()
19-
}
20-
res.status(422).json({ errors: errors.array() })
21-
}
22-
}
13+
// export const validate = (validations: any) => {
14+
// return async (req: Request, res: Response, next: NextFunction) => {
15+
// await Promise.all(validations.map((validation: any) => validation.run(req)))
16+
// const errors = validationResult(req)
17+
// if (errors.isEmpty()) {
18+
// return next()
19+
// }
20+
// res.status(422).json({ errors: errors.array() })
21+
// }
22+
// }

src/routers/eCommerce.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
import express from 'express'
22

3+
import verifyJWT from '../middlewares/jwt'
34
import {
45
findAll,
56
createProduct,
@@ -15,7 +16,7 @@ const router = express.Router()
1516
router.get('/', findAll)
1617
router.get('/:id', findById)
1718
router.get('/:search', findByQuery)
18-
// router.post('/', verifyToken ,createProduct)//admin need token
19+
router.post('/', verifyJWT, createProduct) //admin need token
1920
router.put('/:id', updateProduct)
2021
router.delete('/:id', deleteProduct)
2122

0 commit comments

Comments
 (0)