[Ruby]AbstractController::DoubleRenderError in UsersController#create が解決できません

Question

Ruby(Ruby on Rails)でオリジナルアプリを作成していて、そのアプリにTwitter認証機能を実装しようとしているのですが、AbstractController::DoubleRenderError in UsersController#create というエラーが解消できずユーザー認証画面には進めるのですがログインできません

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

AbstractController::DoubleRenderError in UsersController#create
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".

上記のエラ〜メッセージが出ていて、多分これは「1つのアクションでリダイレクトは1度しか使えない」ということを意味しているのだと思うのですが、メールアドレスでのログインとTwitterでのログインどちらかのリダイレクトを削除するとやはり上手くいきません

該当のソースコード

**controllers/users_controller.rb**


def create
      @user = User.new(
        name: params[:name],
        email: params[:email],
        image_name: "pile-up3.jpg",
        password: params[:password]
      )
      if @user.save
        session[:user_id] = @user.id
        redirect_to("/users/#{@user.id}")
      else
        render("users/new")
      end



      
    #twitter認証の
    user = User.find_or_create_from_auth_hash(request.env['omniauth.auth'])#request.env['omniauth.auth']はTwitter認証で得た情報を格納するもの
    if user
      session[:user_id] = user.id
      redirect_to '/posts/index', notice: "ログインしました。"
    else
      redirect_to root_path, notice: "失敗しました。"
    end
        #twitter認証の
     
    end

**model/users.rb**

class User < ApplicationRecord
  # has_secure_passwordメソッドを追加してください
  has_secure_password
  
  validates :name, {presence: true}
  validates :email, {presence: true, uniqueness: true}

  
  has_many :posts
  
  def posts
    return Post.where(user_id: self.id)
  end





#twitter認証の
  def self.find_or_create_from_auth_hash(auth_hash)
    provider = auth_hash[:provider]
    uid = auth_hash[:uid]
    name = auth_hash[:info][:name]
    image_url = auth_hash[:info][:image]
 
    
    self.find_or_create_by(provider: provider,uid: uid) do |user|
      user.username = name
      user.image_url = image_url
    end
   end
   #twitter認証の
  
end

試したこと

1つのアクションでリダイレクトは1回しか使えないということで先述した通りメールアドレスでのログイン機能とTwitterでのログイン機能どちらかのリダイレクトを削除するというのを試してみたのですが、どちらかを消せばエラーは発生しなくなるのですが連携アプリを認証ページから先に進んでもログイン状態にはならないという問題が発生してしまいます

その時のlogは以下の通りです

Started GET "/auth/twitter" for ::1 at 2020-03-07 18:56:30 +0900
Started GET "/auth/twitter" for ::1 at 2020-03-07 18:56:31 +0900
Started GET "/auth/twitter/callback?oauth_token=AIpaIAAAAAABCbykAAABcLRsw28&oauth_verifier=WjekoIOYhumEKZUIkYl7wxbchKjg504y" for ::1 at 2020-03-07 18:56:34 +0900
Processing by UsersController#create as HTML
  Parameters: {"oauth_token"=>"AIpaIAAAAAABCbykAAABcLRsw28", "oauth_verifier"=>"WjekoIOYhumEKZUIkYl7wxbchKjg504y", "provider"=>"twitter"}
  User Load (0.2ms)  SELECT  "users".* FROM "users" WHERE "users"."id" IS NULL LIMIT ?  [["LIMIT", 1]]
  ↳ app/controllers/application_controller.rb:5
   (0.1ms)  begin transaction
  ↳ app/controllers/users_controller.rb:28
  User Exists (0.2ms)  SELECT  1 AS one FROM "users" WHERE "users"."email" IS NULL LIMIT ?  [["LIMIT", 1]]
  ↳ app/controllers/users_controller.rb:28
   (0.1ms)  rollback transaction
  ↳ app/controllers/users_controller.rb:28
  Rendering users/new.html.erb within layouts/application
  Rendered users/new.html.erb within layouts/application (0.8ms)
  User Load (0.2ms)  SELECT  "users".* FROM "users" WHERE "users"."provider" = ? AND "users"."uid" = ? LIMIT ?  [["provider", "twitter"], ["uid", "3300317354"], ["LIMIT", 1]]
  ↳ app/models/user.rb:27
   (0.0ms)  begin transaction
  ↳ app/models/user.rb:27
  User Exists (0.1ms)  SELECT  1 AS one FROM "users" WHERE "users"."email" IS NULL LIMIT ?  [["LIMIT", 1]]
  ↳ app/models/user.rb:27
   (0.0ms)  rollback transaction
  ↳ app/models/user.rb:27
Completed 200 OK in 46ms (Views: 26.1ms | ActiveRecord: 1.7ms)

やはりメールアドレスを利用したログインとTwitterを利用したログインの二つの機能を入れるとしたらアクションを分けなくてはいけないということなのでしょうか？
そもそも、ログイン機能のアクションを複数個作っても大丈夫なのでしょうか（一つにまとめる必要はないのでしょうか？）

また、片方のリダイレクトを削除してTwitter認証に進めたにもかかわらずログインされなかったのはなぜなのでしょうか

お分かりになる方がいらっしゃいましたらご教授ください

Answer 1

すいません、タイトルで回答するだけで終わる質問が多いので手を抜いてしまいました。
パット見、request.env['omniauth.auth'の有無で登録フローが変わるように見えるので、それで対応されるのはいかがでしょうか。ただ、根本的にはコントローラーとアクションを分割したほうがいいと思います。

def create
  if request.env['omniauth.auth'].presence
    # Twitterでのログイン
    user = User.find_or_create_from_auth_hash(request.env['omniauth.auth'])

    if user
      session[:user_id] = user.id
      redirect_to '/posts/index', notice: "ログインしました。"
    else
      redirect_to root_path, notice: "失敗しました。"
    end
  else
    # メールアドレスでのログイン
    @user = User.new(
      name: params[:name],
      email: params[:email],
      image_name: "pile-up3.jpg",
      password: params[:password]
    )
    if @user.save
      session[:user_id] = @user.id
      redirect_to("/users/#{@user.id}")
    else
      render("users/new")
    end
  end
end

Comments

[akairen]

ご回答いただきありがとうございます

アドバイスいただいた通り分割したら問題なく動くようになりました

Answer 2

すべてのredirect_toやrenderの後にreturnを入れたら解消できるかと思います
returnで後続の処理は抜けますので、複数回コールにはあたりません。

Comments

[rhiroe]

その対応をしてしまうとTwitter認証が行われなくなってしまうのでは？