mirror of
https://github.com/discourse/discourse.git
synced 2025-06-14 19:29:21 +08:00
0de08e780b
This improves the error message(s) displayed when an error happens while using a social login to log in / sign up into a Discourse community. Unfortunately, we can't be super precise in the reason behind the error (it can be the user clicked "cancel" during the authorization phase, or any of the plethora of other possible errors) because the reason isn't provided (either for security reasons, or because it's just hard to do so in a reliable & consistent way accross all social logins). The best I could do was to - add the name of the "social login" provider in the error message - tweak the copy a bit for the different cases handle - fix the CSS/HTML to match the one used by the main application In order to show the name of the provider, we use either the "provider" or the "strategy" query parameter, or use the name of the authenticator (if it's the only one enabled). Internal ref - t/153662 --- **BEFORE**  **AFTER** 
42 lines
1.3 KiB
Ruby
42 lines
1.3 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
class Auth::FacebookAuthenticator < Auth::ManagedAuthenticator
|
|
AVATAR_SIZE = 480
|
|
|
|
def name
|
|
"facebook"
|
|
end
|
|
|
|
def display_name
|
|
"Facebook"
|
|
end
|
|
|
|
def enabled?
|
|
SiteSetting.enable_facebook_logins
|
|
end
|
|
|
|
def register_middleware(omniauth)
|
|
omniauth.provider :facebook,
|
|
setup:
|
|
lambda { |env|
|
|
strategy = env["omniauth.strategy"]
|
|
strategy.options[:client_id] = SiteSetting.facebook_app_id
|
|
strategy.options[:client_secret] = SiteSetting.facebook_app_secret
|
|
strategy.options[:info_fields] = "name,first_name,last_name,email"
|
|
strategy.options[:image_size] = {
|
|
width: AVATAR_SIZE,
|
|
height: AVATAR_SIZE,
|
|
}
|
|
strategy.options[:secure_image_url] = true
|
|
},
|
|
scope: "email"
|
|
end
|
|
|
|
# facebook doesn't return unverified email addresses so it's safe to assume
|
|
# whatever email we get from them is verified
|
|
# https://developers.facebook.com/docs/graph-api/reference/user/
|
|
def primary_email_verified?(auth_token)
|
|
true
|
|
end
|
|
end
|