diff --git a/src/OpenIddict.Abstractions/OpenIddictConstants.cs b/src/OpenIddict.Abstractions/OpenIddictConstants.cs index 6f1fc759..1d6e481b 100644 --- a/src/OpenIddict.Abstractions/OpenIddictConstants.cs +++ b/src/OpenIddict.Abstractions/OpenIddictConstants.cs @@ -39,11 +39,6 @@ namespace OpenIddict.Abstractions public const string LogoutRequest = "openiddict-logout-request:"; } - public static class Metadata - { - public const string ExternalProvidersSupported = "external_providers_supported"; - } - public static class Permissions { public static class Endpoints diff --git a/src/OpenIddict.Server/Internal/OpenIddictServerProvider.Discovery.cs b/src/OpenIddict.Server/Internal/OpenIddictServerProvider.Discovery.cs index 820eb8b5..48b86523 100644 --- a/src/OpenIddict.Server/Internal/OpenIddictServerProvider.Discovery.cs +++ b/src/OpenIddict.Server/Internal/OpenIddictServerProvider.Discovery.cs @@ -4,13 +4,11 @@ * the license and the contributors participating to this project. */ -using System.Linq; using System.Threading.Tasks; using AspNet.Security.OpenIdConnect.Primitives; using AspNet.Security.OpenIdConnect.Server; using JetBrains.Annotations; using Newtonsoft.Json.Linq; -using OpenIddict.Abstractions; namespace OpenIddict.Server { @@ -47,11 +45,6 @@ namespace OpenIddict.Server context.Metadata[OpenIdConnectConstants.Metadata.RequestParameterSupported] = false; context.Metadata[OpenIdConnectConstants.Metadata.RequestUriParameterSupported] = false; - context.Metadata[OpenIddictConstants.Metadata.ExternalProvidersSupported] = new JArray( - from provider in context.HttpContext.Authentication.GetAuthenticationSchemes() - where !string.IsNullOrEmpty(provider.DisplayName) - select provider.AuthenticationScheme); - return base.HandleConfigurationRequest(context); } } diff --git a/test/OpenIddict.Server.Tests/Internal/OpenIddictServerProviderTests.Discovery.cs b/test/OpenIddict.Server.Tests/Internal/OpenIddictServerProviderTests.Discovery.cs index 417e9273..cc983174 100644 --- a/test/OpenIddict.Server.Tests/Internal/OpenIddictServerProviderTests.Discovery.cs +++ b/test/OpenIddict.Server.Tests/Internal/OpenIddictServerProviderTests.Discovery.cs @@ -7,11 +7,7 @@ using System.Threading.Tasks; using AspNet.Security.OpenIdConnect.Client; using AspNet.Security.OpenIdConnect.Primitives; -using Microsoft.AspNetCore.Authentication.Cookies; -using Microsoft.AspNetCore.Authentication.Facebook; -using Microsoft.AspNetCore.Authentication.Google; using Newtonsoft.Json.Linq; -using OpenIddict.Abstractions; using Xunit; namespace OpenIddict.Server.Tests @@ -236,23 +232,5 @@ namespace OpenIddict.Server.Tests Assert.False((bool) response[OpenIdConnectConstants.Metadata.RequestParameterSupported]); Assert.False((bool) response[OpenIdConnectConstants.Metadata.RequestUriParameterSupported]); } - - [Fact] - public async Task HandleConfigurationRequest_ExternalProvidersAreCorrectlyReturned() - { - // Arrange - var server = CreateAuthorizationServer(); - - var client = new OpenIdConnectClient(server.CreateClient()); - - // Act - var response = await client.GetAsync(ConfigurationEndpoint); - var providers = ((JArray) response[OpenIddictConstants.Metadata.ExternalProvidersSupported]).Values(); - - // Assert - Assert.DoesNotContain(CookieAuthenticationDefaults.AuthenticationScheme, providers); - Assert.Contains(FacebookDefaults.AuthenticationScheme, providers); - Assert.Contains(GoogleDefaults.AuthenticationScheme, providers); - } } } diff --git a/test/OpenIddict.Server.Tests/Internal/OpenIddictServerProviderTests.cs b/test/OpenIddict.Server.Tests/Internal/OpenIddictServerProviderTests.cs index 130b5f41..4c37ed51 100644 --- a/test/OpenIddict.Server.Tests/Internal/OpenIddictServerProviderTests.cs +++ b/test/OpenIddict.Server.Tests/Internal/OpenIddictServerProviderTests.cs @@ -15,7 +15,6 @@ using AspNet.Security.OpenIdConnect.Client; using AspNet.Security.OpenIdConnect.Extensions; using AspNet.Security.OpenIdConnect.Primitives; using Microsoft.AspNetCore.Authentication; -using Microsoft.AspNetCore.Authentication.Cookies; using Microsoft.AspNetCore.Builder; using Microsoft.AspNetCore.DataProtection; using Microsoft.AspNetCore.Diagnostics; @@ -1418,24 +1417,6 @@ namespace OpenIddict.Server.Tests return next(context); }); - app.UseCookieAuthentication(); - - // Note: the following client_id/client_secret are fake and are only - // used to test the metadata returned by the discovery endpoint. - app.UseFacebookAuthentication(new FacebookOptions - { - ClientId = "16018790-E88E-4553-8036-BB342579FF19", - ClientSecret = "3D6499AF-5607-489B-815A-F3ACF1617296", - SignInScheme = CookieAuthenticationDefaults.AuthenticationScheme - }); - - app.UseGoogleAuthentication(new GoogleOptions - { - ClientId = "BAF437A5-87FA-4D06-8EFD-F9BA96CCEDC4", - ClientSecret = "27DF07D3-6B03-4EE0-95CD-3AC16782216B", - SignInScheme = CookieAuthenticationDefaults.AuthenticationScheme - }); - app.UseOpenIddictServer(); app.Run(context => diff --git a/test/OpenIddict.Server.Tests/OpenIddict.Server.Tests.csproj b/test/OpenIddict.Server.Tests/OpenIddict.Server.Tests.csproj index 1f7e7af1..f23b7d90 100644 --- a/test/OpenIddict.Server.Tests/OpenIddict.Server.Tests.csproj +++ b/test/OpenIddict.Server.Tests/OpenIddict.Server.Tests.csproj @@ -17,9 +17,6 @@ - - -