Skip to content

Commit

Permalink
Merge pull request #55 from sona-is/allow-dynamic-configs
Browse files Browse the repository at this point in the history
Allow dynamic configs
  • Loading branch information
swelham authored Aug 23, 2022
2 parents a6ad0b9 + 5c61d43 commit 954cd51
Showing 1 changed file with 14 additions and 4 deletions.
18 changes: 14 additions & 4 deletions lib/ueberauth/strategy/microsoft.ex
Original file line number Diff line number Diff line change
Expand Up @@ -134,10 +134,20 @@ defmodule Ueberauth.Strategy.Microsoft do
base_options = [redirect_uri: callback_url(conn)]
request_options = conn.private[:ueberauth_request_options].options

case {request_options[:client_id], request_options[:client_secret]} do
{nil, _} -> base_options
{_, nil} -> base_options
{id, secret} -> [client_id: id, client_secret: secret] ++ base_options
request_options =
Keyword.take(request_options, [
:tenant_id,
:client_id,
:client_secret,
:authorize_url,
:token_url,
:request_opts
])

if nil in Keyword.values(request_options) do
base_options
else
request_options ++ base_options
end
end

Expand Down

0 comments on commit 954cd51

Please sign in to comment.