初めて、外部 API を通じてサインアップ時に使い捨てメールアドレスをチェックするプラグインを開発しました。
今のところ意図通りに動作しています…
Check for disposable email
しかし、アプリを再ビルドまたはアップグレードする際に、db.rake で何らかの問題が発生しているようです:
rails aborted!
ActiveRecord::RecordInvalid: Validation failed: Email is not allowed from that email provider. Please use another email address.
(eval):39:in `block (2 levels) in run_file'
/src/lib/tasks/db.rake:222:in `block in <main>'
bin/rails:17:in `<main>'
Tasks: TOP => db:migrate
(See full trace by running task with --trace)
おそらく、この db.rake の行が何らかのデータをシードしようとしていますが、私のプラグインが無効なメールアドレスをチェックして干渉しているため、実行できないのだと思います:
SeedFu.seed(SeedHelper.paths, SeedHelper.filter)
しかし、同じファイル内で、プラグインがシードデータファイルの実行から除外される仕組みがあることがわかりました:
def self.filter
# Allows a plugin to exclude any specified seed data files from running
DiscoursePluginRegistry.seedfu_filter.any? ?
/^(?!.*(#{DiscoursePluginRegistry.seedfu_filter.to_a.join("|")})).*$/ : nil
end
end
では、どうすればよいのでしょうか？そのような処理を行っているプラグインの例はありますか？それとも、この問題を回避するために別の何かを行う必要がありますか？事前に役立つアドバイスがあれば、大変感謝いたします。
blake
2020 年 8 月 31 日午後 9:50
2
表示されるエラーメッセージには、問題を引き起こしている実際のシードデータファイルがリストされていないのですか？
問題の原因となっているシードデータファイルが特定できたら、プラグインで以下のようにフィルタリングできます。
after_initialize do
register_seedfu_filter "00X_seed_filename"
...
end
素晴らしいです。
エラーに関する限り、上記がすべてです。
== 20200819021210 AddUserSelectableColumnToColorSchemes: migrating ============
-- add_column(:color_schemes, :user_selectable, :bool, {:null=>false, :default=>false})
-> 0.0062s
== 20200819021210 AddUserSelectableColumnToColorSchemes: migrated (0.0100s) ===
== 20200819203846 AddColorSchemeIdToUserOptions: migrating ====================
-- add_column(:user_options, :color_scheme_id, :integer)
-> 0.0035s
== 20200819203846 AddColorSchemeIdToUserOptions: migrated (0.0051s) ===========
== 20200820174703 AddPartialTargetIdIndexToReviewables: migrating =============
-- add_index(:reviewables, [:target_id], {:where=>"target_type = 'Post'", :algorithm=>:concurrently, :name=>"index_reviewables_on_target_id_where_post_type_eq_post"})
-> 0.0079s
== 20200820174703 AddPartialTargetIdIndexToReviewables: migrated (0.0091s) ====
rails aborted!
ActiveRecord::RecordInvalid: Validation failed: Email is not allowed from that email provider. Please use another email address.
(eval):39:in `block (2 levels) in run_file'
/src/lib/tasks/db.rake:222:in `block in <main>'
bin/rails:17:in `<main>'
Tasks: TOP => db:migrate
(See full trace by running task with --trace)
私の小さな新規プラグインで問題を引き起こしている正確なファイルを見つけるか、デバッグすることは可能でしょうか？ユーザーのメール設定に関連する何かを疑っています。
以下にさらにデバッグ情報を示します：
Summary
terrapop@terrapop:/var/www/discourse$ d/rails --trace db:migrate RAILS_ENV=developmentraise_validation_error' /usr/local/lib/ruby/gems/2.6.0/gems/activerecord-6.0.3.2/lib/active_record/validations.rb:53:in save!’block in save!' /usr/local/lib/ruby/gems/2.6.0/gems/activerecord-6.0.3.2/lib/active_record/transactions.rb:375:in block in with_transaction_returning_status’transaction' /usr/local/lib/ruby/gems/2.6.0/gems/activerecord-6.0.3.2/lib/active_record/transactions.rb:212:in transaction’with_transaction_returning_status' /usr/local/lib/ruby/gems/2.6.0/gems/activerecord-6.0.3.2/lib/active_record/transactions.rb:318:in save!’save!' /usr/local/lib/ruby/gems/2.6.0/gems/activerecord-6.0.3.2/lib/active_record/persistence.rb:635:in block in update!’block in with_transaction_returning_status' /usr/local/lib/ruby/gems/2.6.0/gems/activerecord-6.0.3.2/lib/active_record/connection_adapters/abstract/database_statements.rb:278:in transaction’transaction' /usr/local/lib/ruby/gems/2.6.0/gems/activerecord-6.0.3.2/lib/active_record/transactions.rb:366:in with_transaction_returning_status’update!' (eval):39:in block (2 levels) in run_file’eval' /usr/local/lib/ruby/gems/2.6.0/gems/seed-fu-2.3.9/lib/seed-fu/runner.rb:46:in block (2 levels) in run_file’block in open' /usr/local/lib/ruby/gems/2.6.0/gems/seed-fu-2.3.9/lib/seed-fu/runner.rb:57:in open’open' /usr/local/lib/ruby/gems/2.6.0/gems/seed-fu-2.3.9/lib/seed-fu/runner.rb:36:in block in run_file’block in transaction' /usr/local/lib/ruby/gems/2.6.0/gems/activerecord-6.0.3.2/lib/active_record/connection_adapters/abstract/transaction.rb:280:in block in within_new_transaction’block (2 levels) in synchronize' /usr/local/lib/ruby/gems/2.6.0/gems/activesupport-6.0.3.2/lib/active_support/concurrency/load_interlock_aware_monitor.rb:25:in handle_interrupt’block in synchronize' /usr/local/lib/ruby/gems/2.6.0/gems/activesupport-6.0.3.2/lib/active_support/concurrency/load_interlock_aware_monitor.rb:21:in handle_interrupt’synchronize' /usr/local/lib/ruby/gems/2.6.0/gems/activerecord-6.0.3.2/lib/active_record/connection_adapters/abstract/transaction.rb:278:in within_new_transaction’transaction' /usr/local/lib/ruby/gems/2.6.0/gems/activerecord-6.0.3.2/lib/active_record/transactions.rb:212:in transaction’run_file' /usr/local/lib/ruby/gems/2.6.0/gems/seed-fu-2.3.9/lib/seed-fu/runner.rb:26:in block in run’each' /usr/local/lib/ruby/gems/2.6.0/gems/seed-fu-2.3.9/lib/seed-fu/runner.rb:25:in run’seed' /src/lib/tasks/db.rake:222:in block in ’block in execute' /usr/local/lib/ruby/gems/2.6.0/gems/rake-13.0.1/lib/rake/task.rb:281:in each’execute' /usr/local/lib/ruby/gems/2.6.0/gems/rake-13.0.1/lib/rake/task.rb:219:in block in invoke_with_call_chain’mon_synchronize' /usr/local/lib/ruby/gems/2.6.0/gems/rake-13.0.1/lib/rake/task.rb:199:in invoke_with_call_chain’invoke' /usr/local/lib/ruby/gems/2.6.0/gems/rake-13.0.1/lib/rake/application.rb:160:in invoke_task’block (2 levels) in top_level' /usr/local/lib/ruby/gems/2.6.0/gems/rake-13.0.1/lib/rake/application.rb:116:in each’block in top_level' /usr/local/lib/ruby/gems/2.6.0/gems/rake-13.0.1/lib/rake/application.rb:125:in run_with_threads’top_level' /usr/local/lib/ruby/gems/2.6.0/gems/railties-6.0.3.2/lib/rails/commands/rake/rake_command.rb:23:in block in perform’standard_exception_handling' /usr/local/lib/ruby/gems/2.6.0/gems/railties-6.0.3.2/lib/rails/commands/rake/rake_command.rb:20:in perform’invoke' /usr/local/lib/ruby/gems/2.6.0/gems/railties-6.0.3.2/lib/rails/commands.rb:18:in ’require' /usr/local/lib/ruby/gems/2.6.0/gems/bootsnap-1.4.8/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:23:in block in require_with_bootsnap_lfi’register' /usr/local/lib/ruby/gems/2.6.0/gems/bootsnap-1.4.8/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:22:in require_with_bootsnap_lfi’require' /usr/local/lib/ruby/gems/2.6.0/gems/activesupport-6.0.3.2/lib/active_support/dependencies.rb:324:in block in require’load_dependency' /usr/local/lib/ruby/gems/2.6.0/gems/activesupport-6.0.3.2/lib/active_support/dependencies.rb:324:in require’
更新:
問題を見つけ、修正しました:
def seed_primary_email
UserEmail.seed do |ue|
ue.id = DiscourseNarrativeBot::BOT_USER_ID
ue.email = "discobot_email"
ue.primary = true
ue.user_id = DiscourseNarrativeBot::BOT_USER_ID
end
end
discourse/plugins/discourse-narrative-bot/db/fixtures/001_discobot.rb で問題が発生していました。
register_seedfu_filter "001_discobot" を追加することで問題が解決しました。
@blake さん、正しい方向へ導いていただきありがとうございます。現在、正常に動作しています。
最後の質問です: 今後、複数のフィルターを登録するにはどうすればよいでしょうか? 次のように?
register_seedfu_filter "001_discobot|002_xxxxxx|003_yyyy"
blake
2020 年 9 月 1 日午後 3:22
5
1行でできる方法もあるかもしれませんが、複数のフィルターを登録するには、複数行を使用できます。
register_seedfu_filter "001..."
register_seedfu_filter "002..."
blake
2020 年 9 月 1 日午後 3:51
6
ただし、この seedfu_filter API は極めて慎重に使用する必要があります！
プラグインに追加すべきではないと思います。もし他のユーザーがそのシードファイルをまだ実行していない場合、プラグインによって discobot が破損する可能性があります。
代わりに、ID が 0 未満のユーザーを除外する方法を検討してください。そうすれば、プラグインがシステムや discobot のメールをチェックしなくなります。これを実現するには、.human_users メソッドを使用できます。
興味深いですね。ご提案いただいた通り、確認して変更いたします。
@blake
以下の行を追加しました:
return unless value.present?
return unless defined? record.id
return unless record.password_validation_required?
これに追加して:
return unless record.should_validate_email_address?
disco_bot は db.migrate 実行時に既に ID が付与されているため、これで動作するようになりました。
さらに、password_validation_required バリデーターを追加する必要がありました。そうしないと、ログイン中やユーザーアバターを変更する際にもプラグインが発火してしまっていたためです。
これで、sign_up フォームにのみバインドされるようになり、seedfu フィルターは不要になりました。