IntenseWebs/grafana
3
0
Fork 0
mirror of https://github.com/grafana/grafana.git synced 2025-02-25 18:55:37 -06:00
Code Issues Packages Projects Releases Wiki Activity

Auth: Remove unnecessary field from Generic OAuth UI (#81565)

Browse Source 
* Remove unnecessary field

* Update interaction event key
This commit is contained in:
Misi 2024-01-30 16:17:38 +01:00 committed by GitHub
parent b48e1f897e
commit 69a1e7b510
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 3 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
public/app/features/auth-config/ProviderConfigForm.test.tsx
View File

@ -113,7 +113,7 @@ describe('ProviderConfigForm', () => {
{ showErrorAlert: false }
);
expect(reportInteractionMock).toHaveBeenCalledWith('grafana_authentication_ssosettings_updated', {
expect(reportInteractionMock).toHaveBeenCalledWith('grafana_authentication_ssosettings_saved', {
provider: 'github',
enabled: true,
});

2
public/app/features/auth-config/ProviderConfigForm.tsx
View File

@ -56,7 +56,7 @@ export const ProviderConfigForm = ({ config, provider, isLoading }: ProviderConf
}
);
reportInteraction('grafana_authentication_ssosettings_updated', {
reportInteraction('grafana_authentication_ssosettings_saved', {
provider,
enabled: requestData.enabled,
});

6
public/app/features/auth-config/fields.tsx
View File

@ -88,7 +88,7 @@ export const sectionFields: Section = {
{
name: 'TLS',
id: 'tls',
fields: ['configureTLS', 'tlsSkipVerifyInsecure', 'tlsClientCert', 'tlsClientKey', 'tlsClientCa'],
fields: ['tlsSkipVerifyInsecure', 'tlsClientCert', 'tlsClientKey', 'tlsClientCa'],
},
],
};
@ -311,10 +311,6 @@ export function fieldMap(provider: string): Record<string, FieldData> {
'If enabled, Grafana will fetch a new access token using the refresh token provided by the OAuth2 provider.',
type: 'checkbox',
},
configureTLS: {
label: 'Configure TLS',
type: 'switch',
},
tlsClientCa: {
label: 'TLS client ca',
description: 'The path to the trusted certificate authority list. Does not applicable on Grafana Cloud.',