【问题标题】:Devise Twitter Omniauth redirects to sign_up设计 Twitter Omniauth 重定向到 sign_up
【发布时间】:2016-11-11 09:39:22
【问题描述】:

我已经使用omniauth-facebook 成功实现了Devise,现在我正在尝试使用Twitter。我几乎将 Facebook 的 Devise settings 复制到了 Twitter 的方法中。但是,在成功批准我的应用程序使用我的 Twitter 帐户后,我被重定向回我的用户注册页面 (http://localhost:3000/users/sign_up)。为什么是这样?

控制台输出

Started GET "/users/auth/twitter/callback?oauth_token=zeIyTgAAAAAAwAQEAAABVcusPxc&oauth_verifier=q24BAAziukc8bF6nnaxuRoouuGaPuoF3" for ::1 at 2016-07-08 14:01:51 -0400
I, [2016-07-08T14:01:51.984997 #44805]  INFO -- omniauth: (twitter) Callback phase initiated.
Processing by Users::OmniauthCallbacksController#twitter as HTML
  Parameters: {"oauth_token"=>"zeIyTgAAAAAAwAQEAAABVcusPxc", "oauth_verifier"=>"q24BAAziukc8bF6nnaxuRoouuGaPuoF3"}
  User Load (0.7ms)  SELECT  "users".* FROM "users" WHERE "users"."provider" = $1 AND "users"."uid" = $2  ORDER BY "users"."id" ASC LIMIT 1  [["provider", "twitter"], ["uid", "248829852"]]
   (0.2ms)  BEGIN
   (0.1ms)  ROLLBACK
Redirected to http://localhost:3000/users/sign_up
Completed 302 Found in 165ms (ActiveRecord: 1.0ms)

控制器/用户/omniauth_callbacks_controller.rb

类用户::OmniauthCallbacksController

    if @user.persisted?
      sign_in_and_redirect @user, :event => :authentication #this will throw if @user is not activated
      set_flash_message(:notice, :success, :kind => "Facebook") if is_navigational_format?
    else
      session["devise.facebook_data"] = request.env["omniauth.auth"]
      redirect_to new_user_registration_url
    end
  end

  def twitter
    @user = User.from_omniauth(request.env["omniauth.auth"])

    if @user.persisted?
      sign_in_and_redirect @user, :event => :authentication #this will throw if @user is not activated
      set_flash_message(:notice, :success, :kind => "Twitter") if is_navigational_format?
    else
      session["devise.twitter_data"] = request.env["omniauth.auth"].except("extra")
      redirect_to new_user_registration_url
    end
  end

  def failure
    redirect_to root_path
  end
end

models/user.rb

class User < ActiveRecord::Base
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :trackable, :validatable,
         :omniauthable, :omniauth_providers => [:facebook, :twitter]

   def self.from_omniauth(auth)
   where(provider: auth.provider, uid: auth.uid).first_or_create do |user|
     user.email = auth.info.email
     user.password = Devise.friendly_token[0,20]
     user.name = auth.info.name   # assuming the user model has a name
     user.image = auth.info.image # assuming the user model has an image
   end
 end

end

config/routes.rb

Rails.application.routes.draw do

  devise_for :users,
    :controllers => { :omniauth_callbacks => "users/omniauth_callbacks" }


   root 'welcome#index'
end

【问题讨论】:

  • 您是否正在做任何验证以确保您的模型中存在电子邮件?
  • 其实,没有。我还没有对验证做任何事情......我的数据库有一个字段,但此时用户模型中没有验证。
  • 我认为你应该给Twitter用户分配一个临时电子邮件,看看会发生什么。
  • 是的,这是因为缺少电子邮件。所以现在我知道它与设计相关......
  • Devise 为某些字段提供默认验证。详情请看我的回答。

标签: ruby-on-rails ruby twitter devise


【解决方案1】:

像这样将调试输出添加到您的 twitter 回调中,以查看阻止事务提交的确切错误:

  def twitter
    @user = User.from_omniauth(request.env["omniauth.auth"])

    if @user.persisted?
      sign_in_and_redirect @user, :event => :authentication #this will throw if @user is not activated
      set_flash_message(:notice, :success, :kind => "Twitter") if is_navigational_format?
    else
      session["devise.twitter_data"] = request.env["omniauth.auth"].except("extra")

      puts @user.errors

      redirect_to new_user_registration_url
    end
  end

众所周知,Twitter 在注册期间不需要电子邮件,因此在对此类帐户执行 oauth 时它可能不会返回值。

对于这个问题,Devise 默认为某些字段提供自动验证。来自Devise source:

  def self.included(base)
    base.extend ClassMethods
    assert_validations_api!(base)

    base.class_eval do
      validates_presence_of   :email, if: :email_required?

要关闭默认的电子邮件存在验证,请将其放入您的 User 模型中:

def email_required?
  false
end

根据您的用例,您可能还想更改默认身份验证密钥以不使用电子邮件:

在 config/initializers/devise.rb:

config.authentication_keys = [ :username ]

更新

使用 Devise 提供的生成器时,user 表可能会在 email 字段上出现 null: false 约束。要删除它,请使用以下命令创建迁移:

change_column :users, :email, :string, :null => true

并更新数据库架构:

bundle exec rake db:migrate

【讨论】:

  • 添加了email_required? 方法...但是现在我得到了一个不同的、愤怒的错误:ActiveRecord::StatementInvalid (PG::NotNullViolation: ERROR: null value in column "email" violates not-null constraint。所以,我猜它没用?
  • 通过迁移从数据库中的users.email 中删除null: false 约束。
【解决方案2】:

我一直在努力解决同样的错误,使用 Github 身份验证,最后通过在 devise.rb 中将范围添加到 config.omniauth,它起作用了!也许它需要范围用户来检索电子邮件和其他信息

config.omniauth :github, ENV['GITHUB_APP_ID'], ENV['GITHUB_APP_SECRET'], scope: 'repo,user'

【讨论】:

    猜你喜欢
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 2013-09-30
    • 1970-01-01
    • 2022-09-28
    • 1970-01-01
    • 2018-12-31
    • 2017-08-13
    相关资源
    最近更新 更多