ユーザー毎に暗号鍵をもった場合の理想的なMVCモデルについて

Question

実現したいこと

ユーザーIDに紐付いた暗号鍵を利用してアプリを作成した場合、どのようにすればRails的というか理想的なMVCモデルが構築ができるのでしょうか？

前提

• Deviseを利用してユーザー管理を行っています。
• ユーザーIDに紐付いた暗号鍵をyamlデータに保存しています。
• 暗号化、復号には、ActiveSupport::MessageEncryptorを利用します。（参考サイト①）
• 多くのモデルで暗号化の必要があるカラムがあります。（個人情報や銀行口座情報などで必須条件）

実現したこと

モデルからコントローラーの変数（current_user.id）を取得。（参考サイト②）
その変数を基に、yamlに保存してある暗号キーを取得して、暗号化、復号を行いました。（参考サイト①）

気になったこと

参考サイト②で「少なくともRailsのルールを基準として考えると、モデルからコントローラーの変数を取得するというアプリのデザインがそもそも間違っている」という記載がありました。
その考え方は、なんとなく分かるのですが・・・

参考サイト

①暗号化・復号の手順（ActiveSupport::MessageEncryptor利用）

A simple way to encrypt and decrypt in Rails 5
https://medium.com/@mayneweb/a-simple-way-to-encrypt-and-decrypt-in-rails-5-9a514645d066

②モデルからコントローラーの変数を取得する

Ruby on Rails - Access controller variable from model
https://stackoverflow.com/questions/2419120/ruby-on-rails-access-controller-variable-from-model

Answer 1

モデルからコントローラーの変数（current_user.id）を取得。（参考サイト②）
その変数を基に、yamlに保存してある暗号キーを取得して、暗号化、復号を行いました。（参考サイト①）

ここはちょっと奇妙な気はします。
モデルには暗号化済みの値を渡す感じのほうがよいのでは。

鍵のファイルを一種のDB...モデルと考えると、それに紐付いたencrypt, decryptのロジックがあって、そこにコントローラが処理を委託するイメージだとどうですか？

モデルは暗号化されたデータを保存するだけで、それ以上は意識しないように書いたほうが良い気がします。

Comments

[no1knows]

アドバイスいただきありがとうございます。
ロジック自体がシンプルならコントローラー側で暗号化、複合処理で良いと思うのですが、今回は複雑なのでファットコントローラーになってしまうのが心配です。

[Ighrs]

なにもコントローラーとモデルの2つでしか書けないわけではありません
暗号化や復号化みたいものはライブラリ（libフォルダ）に分離して単独定義するとかはどうでしょう？

テストも機能単位で行えるので楽ですよ
コントローラーは分離した各種機能とモデルとを紐付ける制御だけ行うイメージです

[no1knows]

Ighrsさん、ありがとうございます。
ライブラリフォルダというのは、そのようなときに使うのですね。
こちらであればRailsのルールに沿った形になりそうです！