Merge pull request #2150 from spotify/shmidt-i/fix-tsc-auth-microsoft

fix: discovery api leftovers
This commit is contained in:
Fredrik Adelöw
2020-08-28 10:44:50 +02:00
committed by GitHub
2 changed files with 9 additions and 10 deletions
+1 -2
View File
@@ -135,8 +135,7 @@ export const apis = (config: ConfigApi) => {
builder.add(
microsoftAuthApiRef,
MicrosoftAuth.create({
backendUrl,
basePath: '/auth/',
discoveryApi,
oauthRequestApi,
}),
);
@@ -30,15 +30,17 @@ import {
BackstageIdentity,
} from '../../../definitions/auth';
import { OAuthRequestApi, AuthProvider } from '../../../definitions';
import {
OAuthRequestApi,
AuthProvider,
DiscoveryApi,
} from '../../../definitions';
import { SessionManager } from '../../../../lib/AuthSessionManager/types';
import { RefreshingAuthSessionManager } from '../../../../lib/AuthSessionManager';
import { Observable } from '../../../../types';
type CreateOptions = {
backendUrl: string;
basePath: string;
discoveryApi: DiscoveryApi;
oauthRequestApi: OAuthRequestApi;
environment?: string;
@@ -70,15 +72,13 @@ class MicrosoftAuth
BackstageIdentityApi,
SessionStateApi {
static create({
backendUrl,
basePath,
environment = 'development',
provider = DEFAULT_PROVIDER,
oauthRequestApi,
discoveryApi,
}: CreateOptions) {
const connector = new DefaultAuthConnector({
backendUrl,
basePath,
discoveryApi,
environment,
provider,
oauthRequestApi: oauthRequestApi,