アカウント有効化の為の条件式authenticated? が真を返して来ない事への質問

前提・実現したいこと

ただいまrailsチュートリアル１１章を学習中なのですが、
サインアップと同時にユーザーにトークンとユーザーのemailを含めた認証用メールを送り、そこに記載されているURLへアクセスするとアカウントが有効になる、という機能が紹介されていたので自分もそれに挑戦したのですが、何度か試して見たのですがうまく行かなかったので質問させて頂きました。

発生している問題

メールに記載されているURLを踏むと↓アクションに飛ぶのですが、条件式の「user.authenticated?(:activation, params[:id])」がtrueを返して来ず、else以下の処理を行ってしまいます。
※user.authenticated?(:activation, params[:id])をコメントアウトすると全てtrueになり、無事処理されます。

  def edit
    user = User.find_by(email: params[:email])
    if user && !user.activated? && user.authenticated?(:activation, params[:id])←ここ
　　   user.activate
      log_in user
      flash[:success] = "Account activated!"
      redirect_to user
    else
      flash[:danger] = "Invalid activation link"
      redirect_to root_url
    end
  end

該当のソースコード

メールを送られたユーザーの情報

<User id: 113, name: "authenticateテスト", email: ＊＊＊（伏せてます）, 
created_at: "2019-07-27 07:47:33", 
updated_at: "2019-07-27 08:29:18", 
password_digest: "$2a$10$LSSLB18esxbEBcdGb5W6TOFReJuZzQIsvbXEw19f9Xd...", 
admin: nil, 
activation_digest: "$2a$10$a2q6NrZbZTWZ0Ep7J.J9XucLxFh199sfQG6D.Q1LjV8zdVCOHUNY2"
activated: false, activated_at: "2019-07-27 08:29:18", 
reset_digest: nil, reset_sent_at: nil>

メールを送る処理

def send_activation_email
    UserMailer.account_activation(self).deliver_now
end

メールのオブジェクト（html形式）

<%= @user.name %>,


Welcome to the Sample App! Click on the link below to activate your account:

<%= link_to "Activate", edit_account_activation_url(@user.activation_token,
                                                    email: @user.email) %>

authenticated?メソッド

def authenticated?(attribute, token)
  digest = self.send("#{attribute}_digest")
  return false if digest.nil?
  BCrypt::Password.new(digest).is_password?(token)
  end

試したこと

authenticatedメソッド内の

BCrypt::Password.new(digest).is_password?(token)

のdigestとtokenの部分に"i788eilvO80XVhwNnomZJA"（トークン）,
"$2a$10$a2q6NrZbZTWZ0Ep7J.J9XucLxFh199sfQG6D.Q1LjV8zdVCOHUNY2"(ダイジェスト)を直接打ち込んでもfalseが返ってくるので、もしかしたら上の文章が間違っているのかもしれないと色々調べたのですが、該当する理由が見つからず質問させて頂きました。

補足情報（FW/ツールのバージョンなど）

rails 5.1.6 ruby2.6.3

y_waiwai

2019/07/27 10:27

どこにサインアップするというはなしなんでしょうか

kingyo0802

2019/07/27 10:50

いえ、単純に何が問題でこのauthenticated?というメソッドがfalseになっているか知りたいだけです。

行動規範の内容に同意します

回答1件

ベストアンサー

コードに不審な点は見受けられません。
(何個か全角空白が混入していますが、実際のコードにも混入している場合動かない筈)

強いていえば、Userクラスにbefore_create :create_activation_digestが存在するかが不透明です

手元でも確かめましたがtokenとdigestがマッチしていませんので
原因としては、create_activation_digestメソッドを

ruby
1def create_activation_digest
2  self.activation_token  = User.new_token
3  self.activation_digest = User.digest(User.new_token) #
4end

このように間違えているか、複数回呼び出している可能性があります。

create_activation_digestメソッド内でactivation_tokenとactivation_digestのペアをログに出力してみてはいかがでしょうか？

投稿2019/07/28 00:25

asm

総合スコア15147

kingyo0802

2019/07/28 08:41

解答ありがとうございます！こちらでソースコードを読み直したところ、before_create :create_activation_digestは存在し def create_activation_digestも特には問題はありませんでした（User.digestの引数にはちゃんとactivation_tokenが入っていました）念の為全角スペースの検索も行ったのですが、特に存在しなかったようなので、多分自分がどこかしらで、スペルミスを行っているのかもしれないと思い、関係しそうなものをもう一度書き直してみたら、なんと、無事動くようになりました！お騒がせさせてしまい申し訳ありませんでした。見直すきっかけをくれた意味で、ベストアンサーにしておきます！　ありがとうございました！

行動規範の内容に同意します