My two cents:

* Left cent: ActionMailer will be magically delicious and asynchronous in Rails 4: https://github.com/rails/rails/pull/6839
* Right cent: I think the simplest thing to do now is just extract the whole delivery into a private method, `delivery_email(user)`. It handles every case we have right now and will definitely not cause issues.