diff --git a/flask_dance/contrib/discord.py b/flask_dance/contrib/discord.py index 728a0fb..0926d58 100644 --- a/flask_dance/contrib/discord.py +++ b/flask_dance/contrib/discord.py @@ -58,7 +58,7 @@ def make_discord_blueprint( scope = scope or ["identify"] authorization_url_params = {"prompt": "consent"} if prompt is None: - authorization_url_params["prompt"] = None + authorization_url_params["prompt"] = "none" discord_bp = OAuth2ConsumerBlueprint( "discord", __name__, diff --git a/tests/contrib/test_discord.py b/tests/contrib/test_discord.py index 4bc9ed4..92f4cdd 100644 --- a/tests/contrib/test_discord.py +++ b/tests/contrib/test_discord.py @@ -39,7 +39,25 @@ def test_blueprint_factory(): assert discord_bp.authorization_url_params["prompt"] == "consent" -def test_blueprint_factory_with_prompt(): +def test_blueprint_factory_with_prompt_consent(): + discord_bp = make_discord_blueprint( + client_id="foo", + client_secret="bar", + scope=["identify", "email"], + redirect_to="index", + prompt="consent", + ) + assert isinstance(discord_bp, OAuth2ConsumerBlueprint) + assert discord_bp.session.scope == ["identify", "email"] + assert discord_bp.session.base_url == "https://discord.com/" + assert discord_bp.session.client_id == "foo" + assert discord_bp.client_secret == "bar" + assert discord_bp.authorization_url == "https://discord.com/api/oauth2/authorize" + assert discord_bp.token_url == "https://discord.com/api/oauth2/token" + assert discord_bp.authorization_url_params["prompt"] == "consent" + + +def test_blueprint_factory_with_prompt_None(): discord_bp = make_discord_blueprint( client_id="foo", client_secret="bar", @@ -54,7 +72,7 @@ def test_blueprint_factory_with_prompt(): assert discord_bp.client_secret == "bar" assert discord_bp.authorization_url == "https://discord.com/api/oauth2/authorize" assert discord_bp.token_url == "https://discord.com/api/oauth2/token" - assert discord_bp.authorization_url_params["prompt"] == None + assert discord_bp.authorization_url_params["prompt"] == "none" def test_load_from_config(make_app):