From 988a712c7736da4a3ca7a7e57c69771e5ab2f52a Mon Sep 17 00:00:00 2001 From: Sam Date: Sun, 24 Mar 2013 17:21:18 -0700 Subject: [PATCH] correct multisite functionality for omniauth --- Gemfile | 2 +- config/initializers/omniauth.rb | 43 +++++++++++++++++++++------------ 2 files changed, 28 insertions(+), 17 deletions(-) diff --git a/Gemfile b/Gemfile index c0ebf522031..a897688f264 100644 --- a/Gemfile +++ b/Gemfile @@ -100,7 +100,7 @@ end group :development do gem 'better_errors' - gem 'binding_of_caller' # I tried adding this and got an occational crash + gem 'binding_of_caller' gem 'librarian', '>= 0.0.25', require: false gem 'pry-rails' end diff --git a/config/initializers/omniauth.rb b/config/initializers/omniauth.rb index 20046027318..32e292f03da 100644 --- a/config/initializers/omniauth.rb +++ b/config/initializers/omniauth.rb @@ -7,31 +7,42 @@ require 'openid_redis_store' Rails.application.config.middleware.use OmniAuth::Builder do provider :open_id, - store: OpenID::Store::Redis.new($redis), - name: 'google', - identifier: 'https://www.google.com/accounts/o8/id', - require: 'omniauth-openid' + :store => OpenID::Store::Redis.new($redis), + :name => 'google', + :identifier => 'https://www.google.com/accounts/o8/id', + :require => 'omniauth-openid' provider :open_id, - store: OpenID::Store::Redis.new($redis), - name: 'yahoo', - identifier: 'https://me.yahoo.com', - require: 'omniauth-openid' + :store => OpenID::Store::Redis.new($redis), + :name => 'yahoo', + :identifier => 'https://me.yahoo.com', + :require => 'omniauth-openid' + # lambda is required for proper multisite support, + # without it subdomains will not function correctly provider :facebook, - SiteSetting.facebook_app_id, - SiteSetting.facebook_app_secret, - scope: "email" + :setup => lambda { |env| + strategy = env['omniauth.strategy'] + strategy.options[:client_id] = SiteSetting.facebook_app_id + strategy.options[:client_secret] = SiteSetting.facebook_app_secret + }, + :scope => "email" provider :twitter, - SiteSetting.twitter_consumer_key, - SiteSetting.twitter_consumer_secret + :setup => lambda { |env| + strategy = env['omniauth.strategy'] + strategy.options[:consumer_key] = SiteSetting.twitter_consumer_key + strategy.options[:consumer_secret] = SiteSetting.twitter_consumer_secret + } provider :github, - SiteSetting.github_client_id, - SiteSetting.github_client_secret + :setup => lambda { |env| + strategy = env['omniauth.strategy'] + strategy.options[:client_id] = SiteSetting.github_client_id + strategy.options[:client_secret] = SiteSetting.github_client_secret + } provider :browser_id, - name: 'persona' + :name => 'persona' end