I'm following a tutorial for Devise+Omniauth+Rails 4 with multiple providers. My code is very close to the examples at the sourcery tutorial except that I've left off the confirmable module and changed linkedin to google_oauth2.
Everything is working fine until I get to the callback and then rails raises this error:
NameError - undefined local variable or method `provider' for #<OmniauthCallbacksController:0x007fa8312a5298>:
(eval):7:in `twitter'
Here's my OmniauthCallbacksController (nearly identical to the link above):
class OmniauthCallbacksController < Devise::OmniauthCallbacksController
def self.provides_callback_for(provider)
class_eval %Q{
def #{provider}
#user = User.find_for_oauth(env["omniauth.auth"], current_user)
if #user.persisted?
sign_in_and_redirect #user, event: :authentication
set_flash_message(:notice, :success, kind: provider.capitalize) if is_navigational_format?
else
session["devise.#{provider}_data"] = env["omniauth.auth"]
redirect_to new_user_registration_url
end
end
}
end
[:twitter, :facebook, :google_oauth2].each do |provider|
self.provides_callback_for provider
end
end
Please change the following line:
set_flash_message(:notice, :success, kind: provider.capitalize) if is_navigational_format?
To:
set_flash_message(:notice, :success, kind: "#{provider}".capitalize) if is_navigational_format?
Note the curly brackets around #{provider}, and it also needs to be stringified since the argument is passed in as a symbol.
Related
The first time the User is created but it doesn't keep me logged in, even if I set sign_in #user after creating the Customer, the next time I access it does sign me in.
Am I missing something?
class CallbacksController < Devise::OmniauthCallbacksController
def facebook
#user = User.from_omniauth(request.env["omniauth.auth"])
if resource.id.blank?
sign_in resource, event: :authentication
set_flash_message(:notice, :success, kind: 'Facebook') if is_navigational_format?
# Create customer
Customer.create(:user_id => resource.id, :name => resource.name, :lastname => resource.lastname)
redirect_to new_user_session_path
else
sign_in resource_name, resource
redirect_to profile_path
end
end
end
So I had to add
session["warden.user.user.key"][0] = resource.id
for a weird reason it wasn't adding the ID from the user to the session.
This worked for me but I don't think is the "way" to do it.
I installed Facebook signup with the gem 'Omniauth' and now Users can connect with it. And I use Devise in my website.
I want to know how I have to redirection path like finish sign up after Users are connected with Facebook because they have to fill a form with specific information.
Do I add a new condition in an existing controller file? I created one which names omniauth_callbacks_controller.rb
and the code is:
class Users::OmniauthCallbacksController < Devise::OmniauthCallbacksController
def facebook
user = User.find_for_facebook_oauth(request.env['omniauth.auth'])
if user.persisted?
sign_in_and_redirect user, event: :authentication
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
end
And I try to do this
if user.persisted?
sign_in_and_redirect user, event: :authentication
set_flash_message(:notice, :success, kind: 'Facebook') if is_navigational_format?
user.thing_you_want_to_check.blank?
redirect_to finish_sign_up_path
else
session['devise.facebook_data'] = request.env['omniauth.auth']
redirect_to new_user_registration_url
end
end
end
add a condition that checks whether the fields have been filled out in your sessions controller create action:
def create
user = User.from_omniauth(env["omniauth.auth"])
session[:user_id] = user.id
if user.thing_you_want_to_check.blank?
redirect_to finish_sign_up_path
else
redirect_to root_url, notice: "Signed In!"
end
end
EDIT using the updated code you provided:
class Users::OmniauthCallbacksController < Devise::OmniauthCallbacksController
def facebook
user = User.find_for_facebook_oauth(request.env['omniauth.auth'])
if user.persisted? && user.some_field.blank?
redirect_to # put the path you want them to go to here
elsif user.persisted?
sign_in_and_redirect user, event: :authentication
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
end
The some_field should be the field they would fill out at the path you're directing them to. If it already exists, you just want them to sign_in and not have to go to that path. That's why we're checking to make sure it's blank.
I'm using devise with my Rails 4 app.
I have the following in my user.rb model:
def send_admin_mail
AdminMailer.new_user_waiting_for_approval(self).deliver
end
def send_user_welcome_mail
AdminMailer.new_user_waiting_for_access(user).deliver
end
The first method sends me an email when a new user registers. The aim is for the second method to send a welcome email to the new user. The first method works. The second raises an issue with 'no_method_error' (undefined method email in my callbacks controller).
My callback controller has the following:
def linkedin
#user = User.find_for_linkedin_oauth(request.env["omniauth.auth"])
if #user.persisted?
#user.send_admin_mail
#user.send_user_welcome_mail
redirect_to root_path, :event => :authentication
# sign_in_and_redirect #user, :event => :authentication #this will throw if #user is not activated
# set_flash_message(:notice, :success, :kind => "LinkedIn") if is_navigational_format?
else
session["devise.linkedin_data"] = request.env["omniauth.auth"]
redirect_to root_path
end
end
My admin_mailer has:
def new_user_waiting_for_approval(user)
#user = user
mail(to: "myemailaddress#gmail.com", from: "myemailaddress#gmail.com",
subject: "Registration Request #{user.first_name} #{user.last_name} <#{user.email}>")
end
def new_user_waiting_for_access(user)
#user = user
mail(to: user.email, from: "myemailaddress#gmail.com", subject: "Welcome to Co #{user.first_name}")
end
I'm wondering whether I need to replace user with current_user to make use of the devise method or maybe whether some variation on {} and/or "" is required around the user.email in the second method?? I've tried a few different variations and combinations but haven't had any success.
Thank you for your help.
def send_user_welcome_mail
AdminMailer.new_user_waiting_for_access(user).deliver
end
should be replaced with
def send_user_welcome_mail
AdminMailer.new_user_waiting_for_access(self).deliver
end
user -> self
Hi I just setup omniauth for Facebook and Google in my app by following this tutorial : http://sourcey.com/rails-4-omniauth-using-devise-with-twitter-facebook-and-linkedin/
Here's my OmniauthCallbacksController :
class OmniauthCallbacksController < Devise::OmniauthCallbacksController
def self.provides_callback_for(provider)
class_eval %Q{
def #{provider}
#user = User.find_for_oauth(env["omniauth.auth"], current_user)
if #user.persisted?
sign_in_and_redirect #user, event: :authentication
set_flash_message(:notice, :success, kind: "#{provider}".gsub(/_/," ").split[0...1].join(' ').capitalize) if is_navigational_format?
else
session["devise.#{provider}_data"] = env["omniauth.auth"]
redirect_to new_user_registration_url
end
end
}
end
[:google_oauth2, :facebook].each do |provider|
provides_callback_for provider
end
def after_sign_in_path_for(resource)
#if resource.email_verified?
super resource
#else
# finish_signup_path(resource)
#end
end
# def google_oauth2
# raise request.env["omniauth.auth"]
# end
end
I want to redirect all the users who have signed up with facebook or google (Not sign in, sign up for a new account) to their user profile page with a notice asking them to change their username because I set the email as their username on signup. How can I do this?
after this line sign_in_and_redirect #user, event: :authentication
You can add a cookies value that this user is signed up using omniauth.
cookies[:oA] = true
In the after_sign_in_path_for(resource) method check if the value of the cookie is true, add a path username_path(resource) for example.
Newbie here , i have omniauth-facebook setup in an application , every thing working fine , my only issue i want to redirect to another page after the authentication ,but i can't quite figure it out .
my first solution was to add a route to match the auth/facebook/callback to "sessions#new " it's not working . other thing i try is to add a redirect in the controller , not working also , what is the proper to redirect to a specific page ?
controller
class Users::OmniauthCallbacksController < Devise::OmniauthCallbacksController
def facebook
# You need to implement the method below in your model (e.g. app/models/user.rb)
#user = User.find_for_facebook_oauth(request.env["omniauth.auth"], current_user)
if #user.persisted?
sign_in_and_redirect , :event => :authentication #this will throw if #user is not activated
set_flash_message(:notice, :success, :kind => "sign in successfuly") if is_navigational_format?
else
session["devise.facebook_data"] = request.env["omniauth.auth"]
redirect_to new_user_registration_url
end
end
end
routes
devise_for :users, :controllers => { :omniauth_callbacks => "users/omniauth_callbacks" }
thanks for any help .
When your get user by find_for_facebook_oauth method then you can login with that user and use redirect_to for your desired path. You can do something like:
class Users::OmniauthCallbacksController < Devise::OmniauthCallbacksController
def facebook
# You need to implement the method below in your model (e.g. app/models/user.rb)
#user = User.find_for_facebook_oauth(request.env["omniauth.auth"], current_user)
if #user.persisted?
sign_in(#user)
redirect_to desired_path, notice: 'Signed in successfully.'
set_flash_message(:notice, :success, :kind => "sign in successfuly") if is_navigational_format?
else
session["devise.facebook_data"] = request.env["omniauth.auth"]
redirect_to new_user_registration_url
end
end
end