가입 시 외부 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) 줄에서 데이터를 시드(seed)하려고 하지만, 제 플러그인이 잘못된 이메일을 검사하여 방해하기 때문인 것 같습니다.
하지만 같은 파일에서 플러그인이 실행될 시드 데이터 파일에서 제외될 수 있다는 것을 발견했습니다:
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
8월 31, 2020, 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)
제 작은 신규 플러그인에 문제를 일으키는 정확한 파일이 무엇인지 파악하거나 디버깅할 가능성이 있을까요? 이메일로 사용자를 설정하는 모든 부분이 의심됩니다.
여기 더 많은 디버그 정보가 있습니다:
요약
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
9월 1, 2020, 3:22오후
5
한 줄로 처리하는 방법이 있을 수도 있지만, 여러 필터를 등록하려면 여러 줄을 사용할 수 있습니다.
register_seedfu_filter "001..."
register_seedfu_filter "002..."
blake
9월 1, 2020, 3:51오후
6
이 seedfu_filter API는 정말 신중하게 사용해야 합니다!
해당 시드 파일을 아직 실행하지 않은 다른 사용자가 있다면, 플러그인에 이 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 필터가 더 이상 필요하지 않습니다.