itsanytime/registeruser

GitHub: RegisterUser

http://localhost:4000/user http://localhost:4000/create-user using Robo3T GUI

added db connection in app.js (mongoose):

const mongoose = require('mongoose');

// connect db mongoose.connect("mongodb://localhost:27017/auth", { useUnifiedTopology: true, useCreateIndex: true, useNewUrlParser: true });

mongoose.connection.on("error", console.error); mongoose.connection.on("open", function(){ console.log("Database connected ..."); })

prepare DB

---

new file: user.js in new folder models/:

const mongoose = require("mongoose");

const UserSchema = mongoose.Schema({ name: { type: String, required: true }, email: { type: String, required: true }, password: { type: String, required: true } })

module.exports = mongoose.model("User", UserSchema);

---

New file welcome.hbs

Welcome ...

add in routes.js

const bcrypt = require("bcrypt"); const User = require("./models/User");

in routes error handling:

switch

    if(errors){
        req.session.errors = errors;        
        res.redirect('/user');
    } else {
        res.redirect('/create-user')
    }
});

to

    if (errors.length) {
        req.session.errors = errors;
        res.redirect('/user');
    } else {
        // res.redirect('/create-user')
        bcrypt.hash(req.body.password, 10, (err, hash) => {
            if (err) {
                res.json({
                    error: err
                })
            } else {
                const user = new User({
                    name: req.body.name,
                    email: req.body.email,
                    password: hash
                });
                user.save().then(result => {
                    const userName = result.name;
                    res.render("welcome", {userName})
                }).catch(err => {
                    res.json({
                        error: err
                    })
                })
            }
        })
    }
});

in welcome.hbs change stuff to:

Welcome {{userName}}

// debugging works with no .vscode file // mark some stop points in app.js and press F5 -> type node.js // debug output in "debug console"

avoid doubled emails:

in routes: add User.find.... and move bcrypt stuff into else plus change the catch error

    User.find({email: req.body.email}).exec().then(user => {
                if(user.length >= 1) {
                    return res.status(409).json({  // test without return
                        message: "Mail exists" // {"message":"Mail exists"} if user mail is already in database
                    })
                } 
                .
                .
                .
                    .catch(err => {
                            // res.json({
                                // error: err
                                res.status(500).json({error:err})
                            // })
                        }

complete code to avoid doubled emails:

{
        // res.redirect('/create-user')
        // part to avoid doubled email registrations
        User.find({email: req.body.email}).exec().then(user => {
            if(user.length >= 1) {
                return res.status(409).json({  // test without return
                    message: "Mail exists" // {"message":"Mail exists"} if user mail is already in database
                })
            } else {
                bcrypt.hash(req.body.password, 10, (err, hash) => {
                    if (err) {
                        res.json({
                            error: err
                        })
                    } else {
                        const user = new User({
                            name: req.body.name,
                            email: req.body.email,
                            password: hash
                        });
                        user.save().then(result => {
                            const userName = result.name;
                            res.render("welcome", {userName})
                        }).catch(err => {
                            // res.json({
                                // error: err
                                res.status(500).json({error:err})
                            // })
                        })
                    }
                })
            }
        })
    }

Test output on page: {"message":"Mail exists"} -> because user mail is already in database