Skip to content

Commit

Permalink
Refactor: redirect url
Browse files Browse the repository at this point in the history
  • Loading branch information
kmc7468 committed Feb 4, 2025
1 parent 7665b97 commit 2d5c367
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 10 deletions.
7 changes: 1 addition & 6 deletions src/routes/users.ts
Original file line number Diff line number Diff line change
Expand Up @@ -60,11 +60,6 @@ router.get("/resetProfileImg", userHandlers.resetProfileImgHandler);
router.get("/getBanRecord", userHandlers.getBanRecordHandler);

// 회원 탈퇴를 요청합니다.
router.post(
"/withdraw",
query("redirect").optional().isString(),
validatorMiddleware,
userHandlers.withdrawHandler
);
router.post("/withdraw", validatorMiddleware, userHandlers.withdrawHandler);

export default router;
5 changes: 1 addition & 4 deletions src/services/users.ts
Original file line number Diff line number Diff line change
Expand Up @@ -258,9 +258,6 @@ export const getBanRecordHandler: RequestHandler = async (req, res) => {

export const withdrawHandler: RequestHandler = async (req, res) => {
try {
const redirectPath = decodeURIComponent(
(req.query.redirect as string | undefined) || "%2F"
); // TODO: Typing or Validation
const { sid } = getLoginInfo(req);

const user = await userModel.findOne({ _id: req.userOid, withdraw: false });
Expand All @@ -284,7 +281,7 @@ export const withdrawHandler: RequestHandler = async (req, res) => {
await user.save();

// 로그아웃 처리
const redirectUrl = new URL(redirectPath, req.origin).href;
const redirectUrl = new URL("/mypage?withdraw=true", req.origin).href;
const ssoLogoutUrl =
ssoClient?.getLogoutUrl(sid, redirectUrl) ?? redirectUrl;
logout(req);
Expand Down

0 comments on commit 2d5c367

Please sign in to comment.