From 1cf529b134340aaaea890c38af94ba73cc69ca84 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Javi=20Mart=C3=ADn?= Date: Fri, 29 Mar 2024 00:59:13 +0100 Subject: [PATCH] Make Devise find the strategy class automatically Since we're already setting `wordpress_oauth2` using the `option :name` command in the `OmniAuth::Strategies::Wordpress` class, Devise can automatically find the strategy. However, it wasn't working because we were passing a string instead of a symbol. --- config/initializers/devise.rb | 1 - lib/omni_auth/strategies/wordpress.rb | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/config/initializers/devise.rb b/config/initializers/devise.rb index 33b1e77cd..778c5035e 100644 --- a/config/initializers/devise.rb +++ b/config/initializers/devise.rb @@ -284,7 +284,6 @@ Devise.setup do |config| config.omniauth :wordpress_oauth2, Rails.application.secrets.wordpress_oauth2_key, Rails.application.secrets.wordpress_oauth2_secret, - strategy_class: OmniAuth::Strategies::Wordpress, client_options: { site: Rails.application.secrets.wordpress_oauth2_site }, setup: OmniauthTenantSetup.wordpress_oauth2 diff --git a/lib/omni_auth/strategies/wordpress.rb b/lib/omni_auth/strategies/wordpress.rb index dc0ec1f6c..acc1bfa7e 100644 --- a/lib/omni_auth/strategies/wordpress.rb +++ b/lib/omni_auth/strategies/wordpress.rb @@ -5,7 +5,7 @@ require "omniauth-oauth2" module OmniAuth module Strategies class Wordpress < OmniAuth::Strategies::OAuth2 - option :name, "wordpress_oauth2" + option :name, :wordpress_oauth2 option :client_options, {}