CODEWITHSUNDEEP
node.jsexpress
Stepan Rafael
Stepan Rafael

Reputation: 415

Nodejs doesn't redirect to https

I have a node server with express on top. Godaddy is the domain registrar that I use. If I try to connect with www.example.com doesn't work, only with https://www.example.com. This is how the code looks like:

app.use(express_enforces_ssl());
app.enable('trust proxy');
app.use (function (req, res, next) {
  if(req.secure){
    next();
  }else {
    res.redirect('https://' + req.headers.host + req.url);
  }
});

var options = {
    ca: fs.readFileSync('./gd_bundle-g2-g1.crt'),
    key: fs.readFileSync('./server_cert_key.key'),
    cert: fs.readFileSync('./cfbe0f99da37dcae.crt')
};
https = require('https').createServer(options, app);
io = require('socket.io')(https);

However, if I 'ping www.example.com' the server is reached, but not in the browser.

The process is running on port 443 (dedicated port for secured connection).

I have made port forwarding from internal port 443 with output on port 80, this is how domain registrar works, only with port 80.

I don't know how can I fix it. Do you have any idea ?

Upvotes: 0

Views: 130

Answers (2)

Charlie
Charlie

Reputation: 86

Try normalizing the port before listen to it.
In the code below, the port will be normalized according to the app settings. If none were found, it will be defaulted to port 80

function normalizePort(val) {
    var port = parseInt(val, 10);
    if (isNaN(port)) return val;
    if (port >= 0) return port;
    return false;
}

//port 443 if https:// is detected
var port = normalizePort(process.env.PORT || '80');

// for me, I imported ('http') instead of ('https') 
// but it also works on port 443
https.listen(port, function listening() {
    console.log('Listening on %d', server.address().port);
});

Upvotes: 1

s.d
s.d

Reputation: 29436

DNS doesn't usually require ports info. Browser automatically assumes 80 for http scheme and 443 for https scheme.

The app code is only listening on 443. You need to add additional:

require('http').createServer(app).listen(80);

Upvotes: 1

Related Questions