The option to link external auth to registered user in ExternalLogins.razor page in Components/Account/Manage is not working about fluentui-blazor HOT 9 CLOSED

Please supply us with ready-to-run reproduction code in the form of something we can copy/paste, a (zipped) project structure or a GitHub repository.

We do not have capacity to craft or compose a reproduction for every issue that gets raised. In this case I would need to dive into the setup of the external providers first. As you have already done that, help us to help you by sharing what you have. Thanks.

I have Created a repository ready to run. Details are in README. I have added a short video to Issue.
https://github.com/Doc415/Fluent-Issue
Thanks.

from fluentui-blazor.

Please supply us with ready-to-run reproduction code in the form of something we can copy/paste, a (zipped) project structure or a GitHub repository.

We do not have capacity to craft or compose a reproduction for every issue that gets raised. In this case I would need to dive into the setup of the external providers first. As you have already done that, help us to help you by sharing what you have. Thanks.

from fluentui-blazor.

I've created the database locally and ran the migrations. Running the solution now gives:

Am I missing something in UserSecrets perhaps?

from fluentui-blazor.

PS, I see in program.cs that you have builder.Services.AddQuickGridEntityFrameworkAdapter();

You should change that to use our own adapter which works together with the FluentDataGrid

from fluentui-blazor.

I've created the database locally and ran the migrations. Running the solution now gives:

Am I missing something in UserSecrets perhaps?

I have filled dummy id and secret in appsetings.
I should mention it in the Readme
The application runs in my side, and i could not reproduce the error. I couldnt understand if this error appears when starting the application or using the solution for bug?
I am learning asp.net and not very experineced in github communications.
Anything i should do?

from fluentui-blazor.

PS, I see in program.cs that you have builder.Services.AddQuickGridEntityFrameworkAdapter();

You should change that to use our own adapter which works together with the FluentDataGrid

Thanks for the review. I ll fix it.

from fluentui-blazor.

I have filled dummy id and secret in appsetings. I should mention it in the Readme The application runs in my side, and i could not reproduce the error. I am learning asp.net and not very experineced in github communications. Anything i should do?

No worries. We all needed to learn it sometime 😀

I found the error for this. In code you had things like:

 googleOptions.ClientId = configuration["Authentication:Google:ClientId"];
 googleOptions.ClientSecret = configuration["Authentication:Google:ClientSecret"];

where that was specifiedappsettings as GoogleClientId (insteadof ClientId).

Can now debug...

from fluentui-blazor.

Ok, found the issue. Will be changed in the next version of the templates.

For now, you do not need to change the /Account/Manage/ExternalLogins page (but you can if that works for you).

The fix will be made where the form post is handled, which is IdentityComponentsEndpointRouteBuilderExtensions.cs. We already have code in there to fix the /PerformExternalLogin post action. We just need to use the same fix in the /LinkExternalLogin action

1. Add the following at the end of that file:

private static string TemporaryFluentButtonFix(string provider)
{
    // Temporary workaround for FluentButton returning a provider value twice
    // Split the comma-separated list of strings
    var providers = provider.Split(',');

    // Find the value that appears twice in the list
    provider = providers.GroupBy(p => p)
                        .Where(g => g.Count() == 2)
                        .Select(g => g.Key)
                        .First();
    return provider;
}

2. Remove line 39 and further up until the var properties = ... and replace with

 provider = TemporaryFluentButtonFix(provider);

3. In /LinkExternalLogin inset

 provider = TemporaryFluentButtonFix(provider);

before the line var properties = ...

As said, this will be part of the new templates.

Closing this as a solution is provided an will be fixed in next release

from fluentui-blazor.

Ok, found the issue. Will be changed in the next version of the templates.

For now, you do not need to change the /Account/Manage/ExternalLogins page (but you can if that works for you).

The fix will be made where the form post is handled, which is IdentityComponentsEndpointRouteBuilderExtensions.cs. We already have code in there to fix the /PerformExternalLogin post action. We just need to use the same fix in the /LinkExternalLogin action

1. Add the following at the end of that file:

private static string TemporaryFluentButtonFix(string provider)
{
    // Temporary workaround for FluentButton returning a provider value twice
    // Split the comma-separated list of strings
    var providers = provider.Split(',');

    // Find the value that appears twice in the list
    provider = providers.GroupBy(p => p)
                        .Where(g => g.Count() == 2)
                        .Select(g => g.Key)
                        .First();
    return provider;
}

2. Remove line 39 and further up until the var properties = ... and replace with

 provider = TemporaryFluentButtonFix(provider);

3. In /LinkExternalLogin inset

 provider = TemporaryFluentButtonFix(provider);

before the line var properties = ...

As said, this will be part of the new templates.

Closing this as a solution is provided an will be fixed in next release

Thank you very much

from fluentui-blazor.