Email Verification for Imported Accounts #457
Replies: 2 comments 15 replies
-
|
Assuming you imported the accounts with unverified status, try this patch: diff --git a/lib/rodauth/features/verify_account.rb b/lib/rodauth/features/verify_account.rb
index a27b857..f5e3b5c 100644
--- a/lib/rodauth/features/verify_account.rb
+++ b/lib/rodauth/features/verify_account.rb
@@ -60,6 +60,7 @@ module Rodauth
)
internal_request_method(:verify_account_resend)
+ internal_request_method(:setup_account_verification)
internal_request_method
route(:verify_account_resend) do |r|And this code: require 'rodauth'
rodauth = Rodauth.lib do
enable :verify_account
# other rodauth configuration here
end
DB[:accounts].select_order_map(:id).each do |id|
rodauth.setup_account_verification(account_id: id)
end If that works, I can commit the patch. I haven't tried it, so it may not work. |
Beta Was this translation helpful? Give feedback.
-
|
Thank you for your reply. I tried adding that line to the appropriate file in my rodauth installation, but it yields the following error: |
Beta Was this translation helpful? Give feedback.
-
|
Looks like more is needed. This diff includes a spec, so it hopefully will work (or at least be less broken): diff --git a/lib/rodauth/features/internal_request.rb b/lib/rodauth/features/internal_request.rb
index c7224a0..3985e49 100644
--- a/lib/rodauth/features/internal_request.rb
+++ b/lib/rodauth/features/internal_request.rb
@@ -247,6 +247,11 @@ module Rodauth
_handle_remember(request)
end
+ def _handle_setup_account_verification(_)
+ @account = account_ds(session_value).first
+ setup_account_verification
+ end
+
def _handle_account_id_for_remember_key(request)
load_memory
raise InternalRequestError, "invalid remember key"
diff --git a/lib/rodauth/features/verify_account.rb b/lib/rodauth/features/verify_account.rb
index a27b857..f5e3b5c 100644
--- a/lib/rodauth/features/verify_account.rb
+++ b/lib/rodauth/features/verify_account.rb
@@ -60,6 +60,7 @@ module Rodauth
)
internal_request_method(:verify_account_resend)
+ internal_request_method(:setup_account_verification)
internal_request_method
route(:verify_account_resend) do |r|
diff --git a/spec/verify_account_spec.rb b/spec/verify_account_spec.rb
index 812b903..dabe948 100644
--- a/spec/verify_account_spec.rb
+++ b/spec/verify_account_spec.rb
@@ -493,5 +493,15 @@ describe 'Rodauth verify_account feature' do
login(:login=>'foo@example5.com')
page.body.must_include 'Logged In'
+
+ ds = DB[:accounts].where(:email=>'foo@example5.com')
+ account_id = ds.get(:id)
+ ds.update(:status_id=>1)
+ DB[:account_verification_keys].delete
+ app.rodauth.setup_account_verification(:account_id=>account_id)
+
+ link = email_link(/(\/verify-account\?key=.+)$/, 'foo@example5.com')
+ key = link.split('=').last
+ app.rodauth.verify_account(:verify_account_key=>key, :password=>'0123456789').must_be_nil
end
end
|
Beta Was this translation helpful? Give feedback.
-
|
Now we are at: I'm not sure exactly what I'm intended to set for what I assume is the |
Beta Was this translation helpful? Give feedback.
-
https://paste.debian.net/hidden/65d26cf1/
This setup had been working previously, and the only changes I've made to the working project are those discussed in this thread. |
Beta Was this translation helpful? Give feedback.
-
|
Any chance your |
Beta Was this translation helpful? Give feedback.
-
Indeed, but I can't move it from the bottom or else Mail is uninitialized. config.ru: dev = ENV['RACK_ENV'] == 'development'
if dev
require 'logger'
logger = Logger.new($stdout)
end
require 'rack/unreloader'
Unreloader = Rack::Unreloader.new(subclasses: %w'Roda Sequel::Model', logger: logger, reload: dev, autoload: dev){MapleApp}
require_relative 'models'
Unreloader.require('app.rb'){'MapleApp'}
run(dev ? Unreloader : Library.freeze.app)
unless dev
require 'tilt/sass' unless File.exist?(File.expand_path('../compiled_assets.json', __FILE__))
Tilt.finalize!
end
freeze_core = false
#freeze_core = !dev # Uncomment to enable refrigerator
if freeze_core
begin
require 'refrigerator'
rescue LoadError
else
# When enabling refrigerator, you may need to load additional
# libraries before freezing the core to work correctly. You'll
# want to uncomment the appropriate lines below if you run into
# problems after enabling refrigerator.
# rackup -s webrick
#require 'forwardable'
#require 'webrick'
# rackup -s Puma
#require 'yaml'
#Gem.ruby
# Puma (needed for state file)
#require 'yaml'
# Unicorn (no changes needed)
Refrigerator.freeze_core
end
end
case ENV['RACK_ENV']
when 'development'
Mail.defaults do
delivery_method :smtp, :address => "127.0.0.1", :port => 1025 # or whatever port maildev SMTP listens on
end
when 'test'
Mail.defaults do
delivery_method :test # this allows you to check Mail::TestMailer.deliveries for received email in tests
end
end |
Beta Was this translation helpful? Give feedback.
-
|
Your error is that |
Beta Was this translation helpful? Give feedback.
-
|
Thank you. Though I do wonder why I didn't have to explicitly In any case, everything does appear to be working now. Though it has since occurred to me that importing the accounts as "Verified" and forcing a password reset essentially accomplishes the same thing, and rodauth does gracefully handle non-existent passwords. |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Hello,
I am migrating an old WordPress site to Roda/Sequel/Rodauth, and I have to migrate a bunch of user accounts. In the process, I'd like to enforce email verification and password resets for these old accounts to weed out the many, many bots.
I was sort of hoping that merely importing the usernames/email addresses to the
accountstable would allow the framework to gracefully handle the rest by virtue of no other information being present in the database, but unfortunately that is not so. The first roadblock is that email verification cannot be enforced because the system is unable to "resend" a verification email that it never sent in the first place.Is there any advice on the best way to accomplish this?
Thank you for your time.
Beta Was this translation helpful? Give feedback.
All reactions