Digest::SHA512でpasswordをハッシュ化してDBに保存したい

Question

やりたいこと

現在個人アプリを作成しており、勉強のためユーザー登録の際に入力されたpasswordを、bcryptなどのgemは利用せずにDigest::SHA512メソッドを使用してDBにハッシュ化してから登録したいと考えています。

createアクションでUser.saveする前に1度user_paramsをハッシュに変換して、passwordを取り出し、SHAメソッドでハッシュ化してから登録したいと考えているのですが、その方法が分からず詰まっています。

現在のソースコード

users_controller.rb

  def create

    @user = User.new(user_params)
    if @user.save
      flash[:success] = 'ユーザー登録が完了しました'
      redirect_to root_path
    else
      render :new
    end
  end

  private

  def user_params
    params.require(:user).permit(:name, :email, :password)
  end
end

new.html.erb

<div class="container">
  <div class="row">
    <div class="col-md-6 col-md-offset-3">
      <div class="text-left h1">ユーザー登録</div>
        <%= form_with model: @user, url: users_path do |f| %>
        <% if @user.errors.any? %>
          <ul>
            <% @user.errors.full_messages.each do |message| %>
              <li><%= message %></li>
            <% end %>
          </ul>
        <% end %>
          <div class = 'form-group'>
            <%= f.label :name %>
            <%= f.text_field :name, class: 'form-control' %>
          </div>
          <div class = 'form-group'>
            <%= f.label :email %>
            <%= f.email_field :email, class: 'form-control' %>
          </div>
          <div class = 'form-group'>
            <%= f.label :password_digest %>
            <%= f.password_field :password, class: 'form-control' %>
          </div>
            <%= f.submit "登録", class: 'btn btn-primary' %>
        <% end %>
      </div>
    </div>
  </div>
</div>

一通りの実装の流れや考え方などについてご教授いただけると大変助かります。
よろしくお願いいたします。

Answer 1

Railsのhas_secure_passwordの実装が参考になると思います。

気をつけるべきは「認証時に単純な文字列比較を行わない」ことです。
単純な比較を行った場合、タイミング攻撃によってパスワード(のハッシュ)を解析される恐れがあります。
ActiveSupport::SecurityUtils.secure_compareなどで定数時間で返るようにした方が堅牢です。

まぁ、そもそもSHA512では速すぎて弱いらしい事は頭の隅に置いといた方がいいでしょう。

Comments

[Shokichi]

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

今回は個人アプリ作成でユーザー登録、ログイン等の実装の際に「入力されたパスワードを暗号化してDBに保存する」流れをDeviceやbcryptを使用せず実装したいと考えていますが
今後はセキュリティの面もしっかり考慮していきたいと思っていますので、
SHA512ではセキュリティ面が弱いということは覚えておきたいと思います。