API Strings are not passing through console.log in Express

Question

I am doing the Express crash course, and my first clue that something was wrong is when I ran

app.listen(PORT, () => console.log('Server started on port ${PORT}'));

and it didn't pass the port number through. the problem persists in this area where it simply crashes the server:

const logger = (req, res, next) =>{
console.log('${req.protocol}://${req.get('host')}${req.originalUrl}');
next();
};

Altogether now, here is the full index.js:

const express = require('express');
const path = require('path');
const members = require('./Members');
const app = express();

const logger = (req, res, next) =>{
console.log('${req.protocol}://${req.get('host')}${req.originalUrl}');
next();
};

app.use(logger);

app.get('/api/members', (req, res)=> {
res.json(members);

});

app.use(express.static(path.join(__dirname, 'public')));

const PORT = process.env.PORT || 5000;

app.listen(PORT, () => console.log('Server started on port ${PORT}'));

The error:

   console.log('${req.protocol}://${req.get('host')}${req.originalUrl}');
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

    SyntaxError: missing ) after argument list
        at Module._compile (internal/modules/cjs/loader.js:723:23)
        at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
        at Module.load (internal/modules/cjs/loader.js:653:32)
        at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
        at Function.Module._load (internal/modules/cjs/loader.js:585:3)
        at Function.Module.runMain (internal/modules/cjs/loader.js:831:12)
        at startup (internal/bootstrap/node.js:283:19)
        at bootstrapNodeJSCore (internal/bootstrap/node.js:622:3)

pzaenger · Accepted Answer · 2019-08-03 13:27:19Z

1

Template strings do use back-ticks:

Template literals are enclosed by the back-tick (``)

Therefore

console.log('${req.protocol}://${req.get('host')}${req.originalUrl}');

should be

console.log(`${req.protocol}://${req.get('host')}${req.originalUrl}`);

and

app.listen(PORT, () => console.log('Server started on port ${PORT}'));

should be

app.listen(PORT, () => console.log(`Server started on port ${PORT}`));

answered Aug 3, 2019 at 13:27

pzaenger

12.1k3 gold badges48 silver badges55 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Junaid Babatunde · Accepted Answer · 2019-08-03 13:31:06Z

0

This is because you are using '' which is for an arbitral string... for the port to be displayed, use ``

answered Aug 3, 2019 at 13:31

Junaid Babatunde

2182 silver badges9 bronze badges

1 Comment

mrtunes Over a year ago

Ok this grave character is new to me. Thank you!

Collectives™ on Stack Overflow

API Strings are not passing through console.log in Express

2 Answers 2

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related