aguafrommars / dynamicauthproviders Goto Github PK

Hi, when I am trying to generate database using the commands given in README.md file it generates error as below:

how can I resolve this issue ?

When redis is running in clustered mode, the syntax of the keys need to be changed to make sure all data is stored at the same shard

Describe the bug
When using a redis cluster as the store some operations fails.

To Reproduce
1 setup a redis cluster, the hosted version on Azure can be used
2 configure the connection
3 try to add multiple providers, change soe of them etc
4 exveptions are thrown

Expected behavior
no eceptions

This is what I needed to do to be able to pull in your project into our solution.
It was not possible for us to implement your EF-Core nuget package, since it required us to inherit from your context & baseclass and we don't want to add a nuget package to our shared project that contains our simple objects (POCO)... it just adds too much dependencies which we don't need and want.

The MetadataAddress must use HTTPS unless disabled for development by setting RequireHttpsMetadata=false.
System.InvalidOperationException: The MetadataAddress must use HTTPS unless disabled for development by setting RequireHttpsMetadata=false.
at Microsoft.AspNetCore.Authentication.WsFederation.WsFederationPostConfigureOptions.PostConfigure(String name, WsFederationOptions options)
at Aguacongas.AspNetCore.Authentication.OptionsMonitorCacheWrapper1.TryAdd(String name, AuthenticationSchemeOptions options) at Aguacongas.AspNetCore.Authentication.NoPersistentDynamicManager1.AddAsync(TSchemeDefinition definition, CancellationToken cancellationToken)
at Aguacongas.AspNetCore.Authentication.PersistentDynamicManager`1.AddAsync(TSchemeDefinition definition, CancellationToken cancellationToken)

Nice package, definitely a problem-solver (I hope). The name of the repository is a bit confusing. Is it a typo maybe or is there a thought behind 'Dymamic'?

The startup specifies Facebook provider support via

            dynamicBuilder
                .AddFacebook()

but the case for the Facebook handler is missing from the helper.

I don't want to use some pre-configured OAuth provider (like GitHub on the sample) and only change the client secret and id. Instead I want to be able to specify all related OAuth configuration settings that are needed like the requested claims, the userinfo endpoint, tokenendpoint etc.
I already changed the Create method on HomeController to receive additional user-posted data with the AuthenticationViewModel viewmodel and inside the controller I am creating new OAuthOptions object and populating it.

OAuthOptions oAuthOptions = new OAuthOptions();
oAuthOptions.Scope.Add("openid");
oAuthOptions.Scope.Add("profile");
oAuthOptions.Scope.Add("email");
oAuthOptions.UserInformationEndpoint = "https://mycustomprovider/oauth2/userinfo";

Right now the scopes are not saved in the database.

System.InvalidOperationException: 'The MetadataAddress must use HTTPS unless disabled for development by setting RequireHttpsMetadata=false.'

To meet my requirements, I need to configure multiple OIDC providers within the Identity Server. Each OIDC provider has a different authority URL, which we need to configure from the database. It would be great if you could provide examples on how to achieve this, or we can include this in the current code.

I'm interested by your feedback.
Do you think it's ready for a 1.0.0 release ?

Please comment