userController.js 코드 리뷰

hunjoonrhee commented 2 weeks ago

userContorller.js 코드 리뷰를 하던 중에 몇 가지 눈에 띄는게 있어서 Issue 올려봅니다.

코멘트 처리된 코드: 183번째 줄 ~ 224 번째 줄 -> 이 코드는 나중에 쓸 코드인가요? 아니면 없는게 좋을 것 같습니다.


const router = express.Router();
dotenv.config();
router.use(
session({
secret: 'your_secret_key',
resave: false,
saveUninitialized: true,
cookie: {
  secure: false, // HTTPS를 사용하면 true로 설정
  httpOnly: true,
  sameSite: 'Lax', // 다른 도메인 간 쿠키 전송을 허용하려면 'none'으로 설정
},
}),
);

router.use(bodyParser.json()); router.use(bodyParser.urlencoded({ extended: true })); router.use(passport.initialize()); router.use(passport.session());


이 부분은 굳이 userController에 있지 않아도 될 코드인것 같아요. 그리고 userController에만 해당하는 코드가 아닌거 같구요, app.js로 옮기는게 어떨까요?

Joel-Hwang commented 2 weeks ago

1,2 동의합니다. book 목록 처리시 같이 적용해 주시겠어요? 아니면 따로 브랜치 만들어서 할까요?

hunjoonrhee commented 2 weeks ago

그럼 제가 booksController 하면서 같이 수정하면 될것 같아요! 감사합니다~!

BookDo7starsTS / bookdo7stars_be

userController.js 코드 리뷰 #7