質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.35%

  • JavaScript

    17500questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • Express

    259questions

    ExpressはNode.jsのWebアプリケーションフレームワークです。 マルチページを構築するための機能セットおよびハイブリッドのWebアプリケーションを提供します。

  • CSRF

    47questions

    クロスサイトリクエストフォージェリ (Cross site request forgeries、CSRF)は、 外部Webページから、HTTPリクエストによって、 Webサイトの機能の一部が実行されてしまうWWWにおける攻撃手法です。

JavaScript: Expressでcsrfがうまく動作しない

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 334

Nippun

score 279

JavaScriptとExpressでcsrfがうまく動作しないです。
うまくいけばejsで書いた部分が表示されるはずなのですが'req.csrfToken() is not a function'と言う表示が出てしまいます。

 npm i mongoose csurf express-session method-override connect-flash --s
で必要なものはインストールしました。

参考にしたサイト

app.js

require('./db/mongo');

const createError = require('http-errors');
const express = require('express');
const path = require('path');
const cookieParser = require('cookie-parser');
const logger = require('morgan');

const session = require("express-session");
const csurf = require("csurf");
const mongoose = require("mongoose");
const flash = require("connect-flash");
const methodOverride = require("method-override");
const bodyParser = require('body-parser');

const indexRouter = require('./routes/index');
const usersRouter = require('./routes/users');

const app = express();

mongoose.connect('mongodb://localhost/practice');

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

app.use(logger('dev'));
app.use(express.json()); 
app.use(bodyParser.json()); 
app.use(express.urlencoded({ extended: false })); 
app.use(bodyParser.urlencoded({ extended: false })); 
app.use(cookieParser());
app.use(express.static(path.join(__dirname, 'public')));

app.use('/', indexRouter);
app.use('/users', usersRouter);

// catch 404 and forward to error handler
app.use(function(req, res, next) {
  next(createError(404));
});

// error handler
app.use(function(err, req, res, next) {
    // set locals, only providing error in development
    res.locals.message = err.message;
    res.locals.error = req.app.get('env') === 'development' ? err : {};

    // render the error page
    res.status(err.status || 500);
    res.render('error');
});


app.use(methodOverride(function(req, res){
    if( req.body && typeof req.body === "object" && "_method" in req.body ){
        const method = req.body._method;
        delete req.body._method;
        return method;
    }
}));

app.use(session({
    secret: 'a',
    resave: false,
    saveUninitialized: false,
    cookie: {
        httpOnly: true,
        secure: true,
        maxage: 1000 * 60 * 30 // 30 minutes
    }
}));
app.use(csurf());
app.use(flash());

module.exports = app;


index.js

router.get('/new', function (req, res) {
    res.render('new', {
        title: 'Add message',
        reqCsrf: req.csrfToken(),
    })
});


index.ejs

<form action="/create" method="post">
    <input type="hidden" name="_csrf" value="<%= reqCsrf %>">
    <p>
        <input type="text" name="title" value="" size="60">
        <% if( errors && errors.title ){ %>
            <strong><%= errors.title.message %></strong>
        <% } %>
    </p>
    <p><textarea name="contents" cols="60" rows="12"></textarea></p>
    <p><button type="submit">Create</button></p>
</form>

どのようにすれば動くようになるでしょうか?
できればコードの大きな改変をせずに動作させるようにしたいです。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • Nippun

    2018/04/17 11:45

    view のindex.jsです。

    キャンセル

  • HayatoKamono

    2018/04/17 11:47

    これのことですか? -> const indexRouter = require('./routes/index');

    キャンセル

  • Nippun

    2018/04/17 11:49

    そうです。viewでなくroutesです。すみませんでした。

    キャンセル

回答 1

checkベストアンサー

+1

参考にされている記事の中盤に「app.jsの編集」という見出しのセクションがありますが、そこにあるコードをもう一度よく確認してみて下さい。

app.use(csurf());
app.use(flash());

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

参考にされているサイトのコードを見ると、このようにcsurfミドルウェアの設定を各ルートの設定よりも先に行なっています。

この順序が逆転してしまうと、csurfミドルウェアの設定が行われていない状態で、各ルートがリクエストを処理することになるので、今回のエラーメッセージが出てしまうと思われます。

動作未確認ですが、一度、順序の変更を試してみて下さい。

また、あわせてこの機会にexpressのmiddlewareの仕組みを確認されることをお勧めいたします。

http://expressjs.com/ja/guide/writing-middleware.html
http://expressjs.com/ja/guide/using-middleware.html

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/04/17 12:07

    動きました!!
    ありがとうございます。
    順序は関係ないと思っていました。
    素晴らしい観察眼ですね!!

    キャンセル

同じタグがついた質問を見る

  • JavaScript

    17500questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • Express

    259questions

    ExpressはNode.jsのWebアプリケーションフレームワークです。 マルチページを構築するための機能セットおよびハイブリッドのWebアプリケーションを提供します。

  • CSRF

    47questions

    クロスサイトリクエストフォージェリ (Cross site request forgeries、CSRF)は、 外部Webページから、HTTPリクエストによって、 Webサイトの機能の一部が実行されてしまうWWWにおける攻撃手法です。