Ensure that localProvider is always the 0th provider listed in the provider selection prompt. (#389)

This commit is contained in:
Chad Roberts 2024-09-07 09:50:06 -04:00 committed by GitHub
parent d231ad19d1
commit eca4a72c8c
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 11 additions and 6 deletions

View File

@ -558,7 +558,12 @@ func getAuthProviders(server string) ([]TypedProvider, error) {
if err != nil {
return nil, fmt.Errorf("attempting to decode the auth provider of type %s: %w", providerType, err)
}
supportedProviders = append(supportedProviders, typedProvider)
if typedProvider.GetType() == "localProvider" {
supportedProviders = append([]TypedProvider{typedProvider}, supportedProviders...)
} else {
supportedProviders = append(supportedProviders, typedProvider)
}
}
}

View File

@ -28,6 +28,11 @@ func Test_getAuthProviders(t *testing.T) {
name: "response ok",
server: setupServer(responseOK),
expectedProviders: []TypedProvider{
&apiv3.LocalProvider{
AuthProvider: apiv3.AuthProvider{
Type: "localProvider",
},
},
&apiv3.AzureADProvider{
AuthProvider: apiv3.AuthProvider{
Type: "azureADProvider",
@ -44,11 +49,6 @@ func Test_getAuthProviders(t *testing.T) {
},
},
},
&apiv3.LocalProvider{
AuthProvider: apiv3.AuthProvider{
Type: "localProvider",
},
},
},
},
{