search

se701g2 / Doto

Todo application for Group 2
https://doto.azurewebsites.net/
MIT License
12 stars 42 forks source link

Logger incorrect variable name #218

Closed Minusome closed 4 years ago

Minusome commented 4 years ago

Bug Summary:

Small typo in passport-setup.js

Currently: const { logging } = require('../common/logging');

Should be: const { logger } = require('../common/logging');

Steps To Reproduce

1) Setup a dev environment 2) Login with Google account 3) Close tab then login again with same user account

Expected behaviour

User successfully logs in.

Observed Behaviour

Tab is frozen with stacktrace:

(node:38662) UnhandledPromiseRejectionWarning: ReferenceError: logger is not defined
    at /Users/tonyliu/Desktop/Projects/Doto/doto-backend/src/config/passport-setup.js:24:17
    at processTicksAndRejections (internal/process/task_queues.js:89:5)

Environment