nodejs/nodebestpractices
1
0
Fork 0
mirror of https://github.com/goldbergyoni/nodebestpractices.git synced 2025-10-31 17:47:26 +08:00
Code Issues Packages Projects Releases Wiki Activity

async await

Browse Source
This commit is contained in:
Yedidya
2021-02-05 00:04:16 +02:00
parent 78bd3104d1
commit ec6d4854eb
1 changed files with 13 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
sections/production/assigntransactionid.md
View File

@ -18,16 +18,16 @@ const app = express();
app.use(rTracer.expressMiddleware());
app.get('/getUserData/{id}', (req, res, next) => {
usersRepo.find(req.params.id)
.then((user) => {
app.get('/getUserData/{id}', async (req, res, next) => {
try {
const user = async usersRepo.find(req.params.id);
// At any point in the app after cls-rtracer middleware was initialized, even when 'req' object doesn't exist, the TransactionId is reachable
const transactionId = rTracer.id();
logger.info(`user ${user.id} data was fetched successfully`, { transactionId });
res.json(user);
})
.catch(err) {
} catch (err) {
// As next step I'd recommand using rTracer.id() from inside the logger component, to prevent from using it all over the code
logger.error(`error while fetching user id ${req.params.id} data`, { transactionId: rTracer.id(), error: err });
next(err);
@ -48,11 +48,6 @@ app.use(rTracer.expressMiddleware({
headerName: 'x-transaction-id'
}));
// The supplied TransactionId will be now the one that was passed from the previous request; if this is the first request in the chain, it will be just generated
const transactionId = rTracer.id();
```