简体   繁体   中英

NodeJS express Post form - action always 404

No matter what link I declare in the action of my form, it always returns an 404 error. if I change the method to get, it doesn't. I want to use post because i'm trying to get the data from my input fields and post it in my mongedb database:

HTML:

<form class="container-fluid searchContainer" action="/active" method="post">
    <div class="col-xs-6">
        <div class="form-group">
            <label for="businessUnit">Business unit</label>
            <input type="text" ng-model="businessUnit" id="businessUnit" name="businessUnit" class="form-control"/>
        </div>
        <div class="form-group">
            <label for="tenderId">Tender ID</label>
            <input type="text" name="tenderID" ng-model="tenderId" id="tenderId" class="form-control"/>
        </div>
    </div>
    <div class="col-xs-6">
        <div class="form-group">
            <label for="tenderDate">Tender Bidding Date</label>
            <input type="date" ng-model="tenderDate" name="tenderDate" id="tenderDate" class="form-control"/>
        </div>
        <div class="form-group">
            <label for="therapyArea">Therapy area</label>
            <input type="text" ng-model="therapyArea" name="therapyArea" id="therapyArea" class="form-control"/>
        </div>

    </div>
    <div class="col-md-12 text-right">
        <input type="submit" id="btnCreate" value="Create" class="btn btn-primary" >
    </div>
</form>

JAVASCRIPT:

var express = require('express');
var path = require('path');
var favicon = require('serve-favicon');
var logger = require('morgan');
var cookieParser = require('cookie-parser');
var bodyParser = require('body-parser');
var routes = require('./routes/index');
var users = require('./routes/users');
var active = require('./routes/active');



var app = express();

// view engine setup
app.set('views', path.join(__dirname, 'views'));
app.set('view engine', 'jade');

// uncomment after placing your favicon in /public
//app.use(favicon(path.join(__dirname, 'public', 'favicon.ico')));
app.use(logger('dev'));
app.use(bodyParser.json());
app.use(bodyParser.urlencoded({
    extended: true
}));
app.use(cookieParser());
app.use(express.static(path.join(__dirname, 'public')));

app.use('/', routes);
app.use('/users', users);
app.use('/active', active);


// catch 404 and forward to error handler
app.use(function(req, res, next) {
    var err = new Error('Not Found');
    err.status = 404;
    next(err);
});

// error handlers



// development error handler
// will print stacktrace
if (app.get('env') === 'development') {
    app.use(function(err, req, res, next) {
        res.status(err.status || 500);
        res.render('error', {
            message: err.message,
            error: err
        });
    });
}

// production error handler
// no stacktraces leaked to user
app.use(function(err, req, res, next) {
    res.status(err.status || 500);
    res.render('error', {
        message: err.message,
        error: {}
    });
});


module.exports = app;


var mongoose = require('mongoose');
mongoose.connect('mongodb://localhost/tenders');

var db = mongoose.connection;

db.on('error', function(err) {
    console.log('connection error', err);
});
db.once('open', function() {
    console.log('connected.');
});


var Schema = new mongoose.Schema({
    TenderID: String,
    TenderDate: String,
    BusinessUnit: String,
    TherapyUnit: String
});



var Tender = mongoose.model('Tender', Schema);


app.post('/active', function(req, res) {

    new Tender({
        TenderID: req.body.tenderID,
        TenderDate: req.body.tenderDate,
        BusinessUnit: req.body.businessunit,
        TherapyUnit: req.body.therapyUnit
    }).save(function(err, doc) {
        if (err) res.json(err);
        else res.send('Succesfully insterted')

    })

});

Because you define your 404 route handler before you define app.post('/active') , express evaluates the 404 rule first and returns it. Your POST /active route never gets evaluated. To fix this you need to give Express your POST /active route before your 404 route (which should always be your last route).

Unless you have an equivalent POST /active route defined in the file ./routes/active , the line

app.use('/active', active);

won't fix this problem even though it is evaluated before your 404 route. You can read more about app.use() vs app.get|post|etc() in the express docs .

Express evaluates rules in the order they are passed to it via use() or get|post|delete|etc()

You have defined get route in your server. Try to change the

app.get('/active',function(req,res){

into

app.post('/active',function(req,res){

In this way, you will respond to a post request from the browser.

The technical post webpages of this site follow the CC BY-SA 4.0 protocol. If you need to reprint, please indicate the site URL or the original address.Any question please contact:yoyou2525@163.com.

 
粤ICP备18138465号  © 2020-2024 STACKOOM.COM