codestates / codestates/NERDchat

[Error Handling] DeprecationWarning

Open
#225 0 comments 0 reactions 1 assignee Claimed by @xinnni View on GitHub
Error Server
Dominant language
No language data
Stars
0
Forks
1
PR merge metrics
No merged PRs in 30d

Description

## 어떤 에러인가요?

* DeprecationWarning: Unhandled promise rejections are deprecated.

```
DeprecationWarning: Unhandled promise rejections are deprecated.
```

## 에러 핸들링 방법

async-await에서 에러 핸들링을 안 해주면 생기는 문제이기 때문에 try, catch로 구문을 감싸 주어야 한다.

```
module.exports = async (req, res) => {
const { id, password } = req.headers;
try {
const userData = await Users.findOne({
where: { userId: id, oauth: "none" },
});

if (!userData)
res.status(404).json({ isLogin: false, message: "user not found" });
else if (!comparePassword(password, userData.password))
res.status(400).json({ isLogin: false });
else {
const { id, avatar, userId, nickname, email, oauth, status } = userData;
console.log(id, avatar, userId, nickname, email, oauth, status);
const accessToken = generateAccess({
id,
avatar,
userId,
nickname,
email,
oauth,
status,
});
const refreshToken = generateRefresh({
id,
avatar,
userId,
nickname,
email,
oauth,
status,
});
const expireDate = new Date(Date.now() + 60 * 60 * 1000 * 24);
console.log("!!!!!!!!!!!!!", accessToken);
res
.cookie("accessToken", accessToken, {
httpOnly: true,
expires: expireDate,
sameSite: "none",
secure: true,
})
.cookie("refreshToken", refreshToken, {
httpOnly: true,
sameSite: "none",
secure: true,
})
.status(200)
.json({
data: {
accessToken,
id,
avatar,
userId,
nickname,
email,
oauth,
status,
},
isLogin: true,
});
}

} catch (err) {
console.log(err);
}
};
```

## 에러 핸들링을 위해 참고한 레퍼런스 링크

[Link](https://stackoverflow.com/questions/52811197/unhandledpromiserejectionwarning-on-async-await-promise)

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.