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

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

ただいまの
回答率

90.38%

  • Ruby

    9924questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails

    9337questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

  • Ruby on Rails 4

    2562questions

    Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

  • Server

    233questions

[Ruby On Rails]renderやredirect_toを使っていないのにAbstractController::DoubleRenderError

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 484

s_diff

score 97

ローカル環境や本番環境では問題ないのですが、staging環境でのみDoubleRenderErrorが出ます。
エラーメッセージの最後にredirect_to(...) and returnとあるのですが、このアクション内にはredirect_toもrenderもないため、別のところに問題があると推測しています。
サーバーが立ち上がらなくなったことがあり、それを復旧した際の処理の何かがおそらく引っかかっているのだと思います。

エラー全文

AbstractController::DoubleRenderError (Render and/or redirect were called multiple times in this action. Please note that you may only call render OR redirect, and at most once per action. Also note that neither redirect nor render terminate execution of the action, so if you want to exit an action after redirecting, you need to do something like "redirect_to(...) and return".):

環境

macOS High Sierra(バージョン10.13.6)
ruby 2.3.1p112 (2016-04-26 revision 54768) [x86_64-darwin17]
Rails 4.2.6

試したこと

以下のようにproduct_controllerのdownloadzipアクションにデバッグを追加してログを確認してみました。

# product_controller.rb

def downloadzip
    @product = Product.find_by(id: params[:id])
    download_cnts = DownloadLog.where(user_id: current_user.id).current_month.count
    plan = current_user.enable_download_plan
    logger.debug("aaaaaaaaaaaaaaaaaaaaaaaaaaaaa")
    return if download_cnts >= plan.download_limit
    logger.debug("bbbbbbbbbbbbbbbbbbbbbbbbbbbb")
    file_path = set_zip_url(@product)
    logger.debug("cccccccccccccccccccccccccccccccc")
    ActiveRecord::Base.transaction do
      logger.debug("dddddddddddddddddddddddddddddddd")
      file_download(file_path, "livesearch")
      logger.debug("eeeeeeeeeeeeeeeeeeeeeeeeeeeeee")
      logger.debug(current_user)
      logger.debug("ffffffffffffffffffffffffffffff")
      logger.debug(@product)
      logger.debug("ggggggggggggggggggggggggggggggggg")
      DownloadLog.create(user_id: current_user.id, product_id: @product.id)
      logger.debug("hhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhh")
    end
  end


staging.logでログを確認すると、
logger.debug("ggggggggggggggggggggggggggggggggg")までは問題ないようですが、logger.debug("hhhhhhhhhhhhhhhhhhhhhhhhhhhhhhhh")でのデバッグがなく、DownloadLog.create(user_id: current_user.id, product_id: @product.id)の部分でエラーが起きているようです。

# staging.log

D, [2019-01-10T03:13:33.310428 #9193] DEBUG -- : ggggggggggggggggggggggggggggggggg
D, [2019-01-10T03:13:33.315289 #9193] DEBUG -- :   ESC[1mESC[36mProduct Load (0.3ms)ESC[0m  ESC[1mSELECT  `products`.* FROM `products` WHERE `products`.`id` = 67 LIMIT 1ESC[0m
D, [2019-01-10T03:13:33.316273 #9193] DEBUG -- :   ESC[1mESC[35m (0.2ms)ESC[0m  SELECT COUNT(*) FROM `download_logs` WHERE `download_logs`.`product_id` = 67
D, [2019-01-10T03:13:35.683743 #9193] DEBUG -- :   ESC[1mESC[36m (0.1ms)ESC[0m  ESC[1mROLLBACKESC[0m
E, [2019-01-10T03:13:35.684035 #9193] ERROR -- : Rendering 500 with exception: undefined method `[]' for nil:NilClass
I, [2019-01-10T03:13:35.947113 #9193]  INFO -- : Completed 500 Internal Server Error in 3226ms (ActiveRecord: 8.6ms)
F, [2019-01-10T03:13:36.198009 #9193] FATAL -- : 
AbstractController::DoubleRenderError (Render and/or redirect were called multiple times in this action. Please note that you may only call render OR redirect, and at most once per action. Also note that neither redirect nor render terminate execution of the action, so if you want to exit an action after redirecting, you need to do something like "redirect_to(...) and return".):
  app/controllers/application_controller.rb:51:in `block (2 levels) in handle_500'
  app/controllers/application_controller.rb:49:in `handle_500'


Rendering 500 with exception: undefined method `[]' for nil:NilClassとありますが、[]を記述していないので、なぜこのエラーができているのかわかりません。model/download_log.rbにもapplication_controller.rbのhandle500にも記述はありませんでした。

# model/download_log.rb

class DownloadLog < ActiveRecord::Base
  belongs_to :user, foreign_key: 'user_id'
  belongs_to :product, foreign_key: 'product_id'

  scope :current_month, -> { where(created_at: Time.current.beginning_of_month..Time.current.end_of_month) }

  before_save :set_dl_cnt_to_product

  def set_dl_cnt_to_product
    self.product.update(dl_cnt: DownloadLog.where(product_id: self.product.id).count + 1)
  end
end
# application_controller.rb

class ApplicationController < ActionController::Base
  protect_from_forgery with: :exception
  rescue_from Exception, with: :handle_500 unless Rails.env.development?

  def handle_500(exception = nil)
    logger.error "Rendering 500 with exception: #{exception.message}" if exception
    ExceptionNotifier.notify_exception(exception, env: request.env, data: {message: "error"})
    respond_to do |format|
      format.html {
        return render template: 'errors/error_500', status: 500, layout: false, content_type: 'text/html'
      }
      format.all {
        return head :internal_server_error
      }
    end
  end
end

また、application_controller.rbのhandle_505内の

return render template: 'errors/error_500', status: 500, layout: false, content_type:

の一行を

render template: 'errors/error_500', status: 500, layout: false, content_type: and return


と変えてみましたがエラーメッセージに変化はありませんでした。

実現したいこと

staging環境でのみAbstractController::DoubleRenderErrorが起こる原因を究明したいです。
ご教示いただけることがあれば、何卒よろしくお願いいたします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • macaron_xxx

    2019/01/10 18:34

    こういうのって、意外と再起動したら直った的なこともありますが、再起動なんか試したりしましたか?

    キャンセル

  • s_diff

    2019/01/10 19:12 編集

    コメント有難うございます。
    再起動はcapistranoでデプロイ時にするようになっていて何度も試しています。

    キャンセル

回答 1

checkベストアンサー

+1

      file_download(file_path, "livesearch")

という部分から推測すると、ここの時点でHTTPのレスポンスを返してしまっているのではないかと思います。
そうだったとすると、この時点でサーバ側の処理は完了していなければならないはずです。

この行を一番最後、トランザクションのブロックよりも後に持ってくると動作しますか?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/01/15 12:01

    回答有難うございます!
    返信が遅れて申し訳ありません。

    この行をトランザクションのブロックよりも後に持ってくると、動作しました。
    そして
    DownloadLog.create(user_id: current_user.id, product_id: @product.id)
    が行われてないようでした。

    調べてみると、model/download_log.rbで良くない書き方のコードがあったため、それを削除すると問題なく動きました。該当のコードは以下です。
    before_save :set_dl_cnt_to_product
    def set_dl_cnt_to_product
    self.product.update(dl_cnt: DownloadLog.where(product_id: self.product.id).count + 1)
    end
    問題になる箇所をご指摘してくださり、本当に有難うございます!

    キャンセル

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

  • ただいまの回答率 90.38%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

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

  • Ruby

    9924questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails

    9337questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

  • Ruby on Rails 4

    2562questions

    Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

  • Server

    233questions