Rails API + React + devise current_api_v1_user.id nil:NilClass

Question

#実現したいこと
Rails API + React + deviseでUserを登録しています。current_api_v1_user.idをTestテーブルに登録したいのですがNoMethodError (undefined method `id' for nil:NilClass):が出てしまいuser_idが登録できません。アドバイスいただけないでしょうか。

#エラー文

Started POST "/api/v1/tests" 
Processing by Api::V1::TestsController#create as HTML
  Parameters: {"title"=>"test", "image"=>"", "body"=>"test", "test"=>{"title"=>"test", "image"=>"", "body"=>"test"}}
Completed 500 Internal Server Error in 1ms (ActiveRecord: 0.0ms | Allocations: 660)

NoMethodError (undefined method `id' for nil:NilClass):

#gemファイル

gem 'devise'
gem 'devise_token_auth'
gem 'rack-cors'

#Userモデル
db/migrate/devise_token_auth_create_users.rb

class DeviseTokenAuthCreateUsers < ActiveRecord::Migration[6.1]
  def change
    
    create_table(:users) do |t|
      ## Required
      t.string :provider, :null => false, :default => "email"
      t.string :uid, :null => false, :default => ""

      ## Database authenticatable
      t.string :encrypted_password, :null => false, :default => ""

      ## Recoverable
      t.string   :reset_password_token
      t.datetime :reset_password_sent_at
      t.boolean  :allow_password_change, :default => false

      ## Rememberable
      t.datetime :remember_created_at

      ## Confirmable
      t.string   :confirmation_token
      t.datetime :confirmed_at
      t.datetime :confirmation_sent_at
      t.string   :unconfirmed_email # Only if using reconfirmable

      ## Lockable
      # t.integer  :failed_attempts, :default => 0, :null => false # Only if lock strategy is :failed_attempts
      # t.string   :unlock_token # Only if unlock strategy is :email or :both
      # t.datetime :locked_at

      ## User Info
      t.string :name
      t.string :nickname
      t.string :image
      t.string :email
      t.text :profile

      ## Tokens
      t.text :tokens

      t.timestamps
    end

    add_index :users, :email,                unique: true
    add_index :users, [:uid, :provider],     unique: true
    add_index :users, :reset_password_token, unique: true
    add_index :users, :confirmation_token,   unique: true
    # add_index :users, :unlock_token,         unique: true
  end
end

#Testモデル
db/migrate/create_tests.rb

class CreateTests < ActiveRecord::Migration[6.1]
  def change
    create_table :tests do |t|
      t.integer :user_id
      t.string :title
      t.string :image
      t.text :body
      t.timestamps
    end
  end
end

#app/models/user.rb

class User < ActiveRecord::Base
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable
  include DeviseTokenAuth::Concerns::User
  has_many :tests, dependent: :destroy
end

#app/models/test.rb

class Test < ApplicationRecord
    belongs_to :user
end

#app/controllers/application_controller.rb

class ApplicationController < ActionController::Base
        include DeviseTokenAuth::Concerns::SetUserByToken
      
        skip_before_action :verify_authenticity_token
        helper_method :current_user, :user_signed_in?

end

#Rails（TestsController)
controllers/api/v1/tests_controller.rb

class Api::V1::TestsController < ApplicationController
    def create
        @test = Test.new(test_params)
        @test.user_id = current_api_v1_user.id
        if @test.save
            render json: { status: 200, data: test }
        else
            render json: { status: 500, message: "テストの作成に失敗しました" }
        end 
    end

    private
    def test_params
        params.require(:test).permit(:title, :image, :body)
    end
end

#SessionsController
controllers/api/v1/auth/sessions_controller.rb

class Api::V1::Auth::SessionsController < ApplicationController
    def index
      if current_api_v1_user
        render json: { is_login: true, data: current_api_v1_user }
      else
        render json: { is_login: false, message: "ユーザーが存在しません" }
      end
    end
end

#routes.rb

Rails.application.routes.draw do
  namespace :api do
    namespace :v1 do
      resources :tests, only: %i[index show new create edit ]

      mount_devise_token_auth_for 'User', at: 'auth', controllers: {
        registrations: 'api/v1/auth/registrations'
      }

      namespace :auth do
        resources :sessions, only: %i[index]
      end
    end
  end
end

#reactからSessionsControllerを叩いたjsonのデータ

Answer 1

current_api_v1_userがちゃんと作られていないのではないでしょうか。undefined method `id' for nil:NilClassとなった場合、大抵

〇〇.id

の〇〇に問題があります。その辺りを重点的に見ていくと解決するのではないかと思います。

Comments

[aaa11]

回答ありがとうございます。SessionsController、routes.rb、reactからSessionsControllerを叩いたjsonのデータを追加しました。SessionsControllerを叩いたcurrent_api_v1_userに中身が入っていることは確認できていますがTestsControllerで呼び出した際current_api_v1_userに中身が入っていないという事ですか？

[yaha4967]

そう思います。

createアクションが問題の箇所ですよね？
そこで
puts "中身あるか#{current_api_v1_user}"
みたいな事をすると確かめられると思います。

[aaa11]

中身は何も表示されなかったのでやはりcurrent_api_v1_userに問題がありそうです。
@active_user = User.find_by(id: session[:user_id])
みたいな感じで定義しても中身がありませんでした。

[yaha4967]

やっぱりそうですよね。

ここらからは私、deviceを使ったことないので分からないですけどもう一つ気になった点があってnamespaceが気になりました。的外れかもしれませんが。

[aaa11]

controllers/api/v1/tests_controller.rbの場合/の部分がnamespaceにあたります。

憶測ですがgem の'devise_token_auth'とroutes.rbの表記方法が原因でcurrent_api_v1_userが空になっているかもしれません。

[yaha4967]

routes.rbをちょっと読んでみて、これも憶測なのですが

mount_devise_token_auth_for 'User', at: 'auth', controllers: {
registrations: 'api/v1/auth/registrations'
}

これってマウント足りてなくないですか？

[aaa11]

無事に解決しました。
React側が原因でcurrent_api_v1_userの中身がなかったみたいです。
色々時間を割いていただきありがとうございました。

Answer 2

current_api_v1_userに中身が入っていなかったのはReact側の実装が原因でした。

#変更前
src/lib/api/test.ts

import client from "lib/api/client"
import Cookies from "js-cookie"

import { CreateTestParams } from "interfaces/index"

export const createTest = (params: CreateTestParams)  => {
  return client.post("/tests", params)

#↓

#変更後
src/lib/api/test.ts

import client from "lib/api/client"
import Cookies from "js-cookie"

import { CreateTestParams } from "interfaces/index"

export const createTest = (params: CreateTestParams)  => {
  return client.post("/tests", params, { headers: {
    "access-token": Cookies.get("_access_token"),
    "client": Cookies.get("_client"),
    "uid": Cookies.get("_uid")
  }})
}

#tests_controller.rb

 def create
     @test = Test.new(test_params)

     if @test.save
         render json: { status: 200, data: @test }
     else
         render json: { status: 500, message: "テストの作成に失敗しました" }
     end 
 end

 private
 def test_params
        params.require(:test).permit(:title, :image, :body).merge(user_id: current_api_v1_user.id)
    end

headers情報を送ってないので中身が無いわけです。