Herokuで何度リスタートしてもexitしてしまいます。

Question

前提・実現したいこと

Herokuで何度リスタートしてもexitしてしまいます。

発生している問題・エラーメッセージ

herokuでデプロイしたものを確認しようとしたところ、application errorが出ていて、
heroku logs　--tails で確認したところビルドやデプロイには問題なさそうですが、プロセスが下記エラーを吐いて止まっているようです。
h12エラーに関してはプロセスが止まっていて、応答がなくタイムアウトしていると今のところ考えています。

リスタートの際のエラーログ
 heroku[web.1]: Restarting
 heroku[web.1]: State changed from up to starting
 heroku[web.1]: Stopping all processes with SIGTERM
 heroku[web.1]: Process exited with status 143

httpリクエスト送信時のエラーログ
at=error code=H12 desc="Request timeout" method=GET path="/" dyno=web.1 connect=0ms service=30000ms status=503 bytes=0 protocol=https

git pushの際にはエラー出ていません。

--------------------------------------------------------------------
GET / のエラー処理に問題があるのではないかとの指摘より、
部分的に抜粋しました。
-------------------------------------------------------------------
app.js

let indexRouter = require('./routes/index');
app.use('/', indexRouter);

// 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');
});


routes/index.js

router.get('/', function(req, res, next) {
  const title = 'title';
  const messageKey = nanoid.nanoid(7)
  SecretMessage.findOne({where: {show: true}})
    .then((secret_message) => {
      let data = {title: title, messageKey: messageKey};
      if (secret_message){
        data.secret_message = secret_message.text;
        secret_message.update({show: false});
      }
      if (req.user) {
        data.user = req.user.displayName;
        Room.findAll({where: {createdBy: req.user.id}
      }).then(rooms => {
        data.rooms = rooms;
        res.render('index', data);
      });
    } else {
      res.render('index', data);
    }
  })
});

試したこと

https://devcenter.heroku.com/ja/articles/request-timeout#timeout-behavior
https://stackoverflow.com/questions/15767685/understanding-heroku-server-status-143

上記のurlなどを参考にh12エラーの解決策としてheroku restartなどを行いましたが、何度restartしてもすぐにプロセスが止まってしまい、調べても解決しませんでした。

追記：
アプリの内容は簡単なwebのチャットアプリのようなものです。
アプリのほうに問題があるのではないかという指摘を受けまして、今さらに調べております。
ただ、コードのどこにエラーがあるか分からず、分かる方法があれば追加でお聞きしたいです。
ローカルでは動いているので、heroku用の記述の中にあるのだとは思うのですが、、

宜しくお願い致します。

Comments

[hoshi-takanori]

heroku にデプロイしたアプリに問題があって起動後すぐにエラーで終了してしまうのだと思いますが、どんなアプリか分からないので回答のしようがありません。

[sudsud]

簡単なwebのチャットアプリのようなものなのですが、すみませんどこのコード（または情報）を乗っければいいのか分からず載せていませんでした。取り敢えず簡単に追記しておきます。

[hoshi-takanori]

必要な情報としては、開発に使った言語やフレームワーク、データベースなどですね。あと、heroku に git push したときのエラーメッセージも。

[sudsud]

かなり情報抜け落ちてました、すみません。
必要情報を追記しときましたのでご確認の方宜しくお願い致します。

[hoshi-takanori]

えっと、言語とフレームワークとデータベースは？？？

[sudsud]

node.js / express / postgreSQL です！
タグのほうにだけ書いてました。分かりにくかったすね、すみません。

[hoshi-takanori]

タグは見落としてました。失礼しました。そして express なんですね。その場合、GET / の処理でエラーになった場合に何も返してなくて timeout してる可能性がありますね。差し支えなければ、その辺のソースも貼ってみてください。

[sudsud]

なるほど、、
該当となるような箇所を抜粋しましたので見ていただければと思います。
私の方でもエラー処理で、間違いまたは足りない箇所がないか改めて精査します。

[hoshi-takanori]

router.get('/', 〜 ) で、SecretMessage.findOne と Room.findAll それぞれに対して then はあるけど catch がないですね。.catch(next) を適切な位置に書いたらエラーが捕まるかも？ また、secret_message.update は非同期ではないですか？
(ちなみに、Promise よりも async/await の方が読みやすいのでお勧めです。→ https://qiita.com/yukin01/items/1a36606439123525dc6d )

[sudsud]

返信遅れて申し訳ありません。
catch文を書いたらいけるようになりました！！！
丁寧に対応していただきありがとうございます！！
（async/awaitも試してみます！）

[wss]

現在私も同様のエラーに遭遇しているのですが、よろしければどのように解決できたか教えていただきたいです…
thenの後に
.catch((error) => {
console.error(error)
})
のようなコードを追加しただけで解消したということでしょうか？

[sudsud]

私の場合はそのようにcatch文を書いただけで解決しました。
参考になればいいのですが、、

[wss]

その後解決したのですが、こちらの原因はherokuのNode.jsのバージョンを指定しておらず、デプロイによってNode.jsのバージョンが上がってしまったことによるものでした。
ちなみにこの場合だとcatch文を追加しても変わらず、タイムアウトのためエラーをcatchすることもできなかったです。
返答ありがとうございました！

Answer 1

自己解決ではないですが、hoshi-takanori様よりいただいたコメントで解決しました。
catch文が抜けていて追加すればどうかというものです。以下にこのメッセージ内容をコピペします。流れに関しては　質問への追記・修正の依頼を参照いただければと思います。

router.get('/', 〜 ) で、SecretMessage.findOne と Room.findAll それぞれに対して then はあるけど catch がないですね。.catch(next) を適切な位置に書いたらエラーが捕まるかも？ また、secret_message.update は非同期ではないですか？
(ちなみに、Promise よりも async/await の方が読みやすいのでお勧めです。→ https://qiita.com/yukin01/items/1a36606439123525dc6d )