Replace slash stripping regexp with trimEnd

Signed-off-by: Johan Haals <johan.haals@gmail.com>
This commit is contained in:
Johan Haals
2021-10-06 16:42:03 +02:00
parent 69b8b6efd4
commit a31afc5b62
10 changed files with 35 additions and 16 deletions
@@ -18,6 +18,7 @@ import { Config, JsonValue } from '@backstage/config';
import { SearchOptions } from 'ldapjs';
import mergeWith from 'lodash/mergeWith';
import { RecursivePartial } from '@backstage/plugin-catalog-backend';
import { trimEnd } from 'lodash';
/**
* The configuration parameters for a single LDAP provider.
@@ -286,7 +287,7 @@ export function readLdapConfig(config: Config): LdapProviderConfig[] {
const providerConfigs = config.getOptionalConfigArray('providers') ?? [];
return providerConfigs.map(c => {
const newConfig = {
target: c.getString('target').replace(/\/+$/, ''),
target: trimEnd(c.getString('target'), '/'),
bind: readBindConfig(c.getOptionalConfig('bind')),
users: readUserConfig(c.getConfig('users')),
groups: readGroupConfig(c.getConfig('groups')),
@@ -15,6 +15,7 @@
*/
import { Config } from '@backstage/config';
import { trimEnd } from 'lodash';
/**
* The configuration parameters for a single Microsoft Graph provider.
@@ -72,10 +73,11 @@ export function readMicrosoftGraphConfig(
const providerConfigs = config.getOptionalConfigArray('providers') ?? [];
for (const providerConfig of providerConfigs) {
const target = providerConfig.getString('target').replace(/\/+$/, '');
const authority =
providerConfig.getOptionalString('authority')?.replace(/\/+$/, '') ||
'https://login.microsoftonline.com';
const target = trimEnd(providerConfig.getString('target'), '/');
const authority = providerConfig.getOptionalString('authority')
? trimEnd(providerConfig.getOptionalString('authority'), '/')
: 'https://login.microsoftonline.com';
const tenantId = providerConfig.getString('tenantId');
const clientId = providerConfig.getString('clientId');
const clientSecret = providerConfig.getString('clientSecret');