paulononato/backstage
1
0
Fork 0
Code Issues Pull Requests Actions 39 Packages Projects Releases Wiki Activity

feat: treat providerInfo as a seperate return value

Browse Source 
Signed-off-by: blam <ben@blam.sh>
This commit is contained in:
blam
2024-02-21 11:43:47 +01:00
parent 6f8d32f9bb
commit 9b810b9a99
10 changed files with 47 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files
plugins/auth-backend-module-aws-alb-provider/api-report.md
+5 -1
View File
@@ -21,7 +21,11 @@ export const awsAlbAuthenticator: ProxyAuthenticator<
issuer: string;
getKey: (header: JWTHeaderParameters) => Promise<KeyObject>;
},
AwsAlbResult
AwsAlbResult,
{
accessToken: string;
expiresInSeconds: number;
}
>;
// @public
plugins/auth-backend-module-aws-alb-provider/src/authenticator.ts
+5 -1
View File
@@ -81,8 +81,12 @@ export const awsAlbAuthenticator = createProxyAuthenticator({
return {
result: {
fullProfile,
accessToken: accessToken,
expiresInSeconds: claims.exp,
},
providerInfo: {
accessToken: accessToken,
expiresInSeconds: claims.exp,
accessToken,
},
};
} catch (e) {
plugins/auth-backend-module-gcp-iap-provider/api-report.md
+3
View File
@@ -18,6 +18,9 @@ export const gcpIapAuthenticator: ProxyAuthenticator<
jwtHeader: string;
tokenValidator: (token: string) => Promise<GcpIapTokenInfo>;
},
{
iapToken: GcpIapTokenInfo;
},
{
iapToken: GcpIapTokenInfo;
}
plugins/auth-backend-module-gcp-iap-provider/src/authenticator.ts
+4 -1
View File
@@ -47,6 +47,9 @@ export const gcpIapAuthenticator = createProxyAuthenticator({
const iapToken = await tokenValidator(token);
return { result: { iapToken } };
return {
result: { iapToken },
providerInfo: { iapToken },
};
},
});
plugins/auth-backend-module-oauth2-proxy-provider/api-report.md
+4 -1
View File
@@ -19,7 +19,10 @@ export const OAUTH2_PROXY_JWT_HEADER = 'X-OAUTH2-PROXY-ID-TOKEN';
// @public (undocumented)
export const oauth2ProxyAuthenticator: ProxyAuthenticator<
unknown,
OAuth2ProxyResult
OAuth2ProxyResult,
{
accessToken: string;
}
>;
// @public
plugins/auth-backend-module-oauth2-proxy-provider/src/authenticator.ts
+9 -2
View File
@@ -33,7 +33,8 @@ export const OAUTH2_PROXY_JWT_HEADER = 'X-OAUTH2-PROXY-ID-TOKEN';
/** @public */
export const oauth2ProxyAuthenticator = createProxyAuthenticator<
unknown,
OAuth2ProxyResult
OAuth2ProxyResult,
{ accessToken: string }
>({
defaultProfileTransform: async (result: OAuth2ProxyResult) => {
return {
@@ -63,7 +64,13 @@ export const oauth2ProxyAuthenticator = createProxyAuthenticator<
return req.get(name);
},
};
return { result };
return {
result,
providerInfo: {
accessToken: result.accessToken,
},
};
} catch (e) {
throw new AuthenticationError('Authentication failed', e);
}
plugins/auth-node/api-report.md
+7 -6
View File
@@ -173,13 +173,13 @@ export function createOAuthRouteHandlers<TProfile>(
): AuthProviderRouteHandlers;
// @public (undocumented)
export function createProxyAuthenticator<TContext, TResult>(
authenticator: ProxyAuthenticator<TContext, TResult>,
): ProxyAuthenticator<TContext, TResult>;
export function createProxyAuthenticator<TContext, TResult, TProviderInfo>(
authenticator: ProxyAuthenticator<TContext, TResult, TProviderInfo>,
): ProxyAuthenticator<TContext, TResult, TProviderInfo>;
// @public (undocumented)
export function createProxyAuthProviderFactory<TResult>(options: {
authenticator: ProxyAuthenticator<unknown, TResult>;
authenticator: ProxyAuthenticator<unknown, TResult, unknown>;
profileTransform?: ProfileTransform<TResult>;
signInResolver?: SignInResolver<TResult>;
signInResolverFactories?: Record<
@@ -538,7 +538,7 @@ export type ProfileTransform<TResult> = (
}>;
// @public (undocumented)
export interface ProxyAuthenticator<TContext, TResult> {
export interface ProxyAuthenticator<TContext, TResult, TProviderInfo> {
// (undocumented)
authenticate(
options: {
@@ -547,6 +547,7 @@ export interface ProxyAuthenticator<TContext, TResult> {
ctx: TContext,
): Promise<{
result: TResult;
providerInfo?: TProviderInfo;
}>;
// (undocumented)
defaultProfileTransform: ProfileTransform<TResult>;
@@ -557,7 +558,7 @@ export interface ProxyAuthenticator<TContext, TResult> {
// @public (undocumented)
export interface ProxyAuthRouteHandlersOptions<TResult> {
// (undocumented)
authenticator: ProxyAuthenticator<any, TResult>;
authenticator: ProxyAuthenticator<any, TResult, unknown>;
// (undocumented)
config: Config;
// (undocumented)
plugins/auth-node/src/proxy/createProxyAuthProviderFactory.ts
+1 -1
View File
@@ -28,7 +28,7 @@ import { ProxyAuthenticator } from './types';
/** @public */
export function createProxyAuthProviderFactory<TResult>(options: {
authenticator: ProxyAuthenticator<unknown, TResult>;
authenticator: ProxyAuthenticator<unknown, TResult, unknown>;
profileTransform?: ProfileTransform<TResult>;
signInResolver?: SignInResolver<TResult>;
signInResolverFactories?: Record<
plugins/auth-node/src/proxy/createProxyRouteHandlers.ts
+4 -4
View File
@@ -29,7 +29,7 @@ import { NotImplementedError } from '@backstage/errors';
/** @public */
export interface ProxyAuthRouteHandlersOptions<TResult> {
authenticator: ProxyAuthenticator<any, TResult>;
authenticator: ProxyAuthenticator<any, TResult, unknown>;
config: Config;
resolverContext: AuthResolverContext;
signInResolver: SignInResolver<TResult>;
@@ -56,7 +56,7 @@ export function createProxyAuthRouteHandlers<TResult>(
},
async refresh(this: never, req: Request, res: Response): Promise<void> {
const { result } = await authenticator.authenticate(
const { result, providerInfo } = await authenticator.authenticate(
{ req },
authenticatorCtx,
);
@@ -68,9 +68,9 @@ export function createProxyAuthRouteHandlers<TResult>(
resolverContext,
);
const response: ClientAuthResponse<{}> = {
const response: ClientAuthResponse<unknown> = {
profile,
providerInfo: {},
providerInfo,
backstageIdentity: prepareBackstageIdentityResponse(identity),
};
plugins/auth-node/src/proxy/types.ts
+5 -5
View File
@@ -19,18 +19,18 @@ import { Request } from 'express';
import { ProfileTransform } from '../types';
/** @public */
export interface ProxyAuthenticator<TContext, TResult> {
export interface ProxyAuthenticator<TContext, TResult, TProviderInfo> {
defaultProfileTransform: ProfileTransform<TResult>;
initialize(ctx: { config: Config }): TContext;
authenticate(
options: { req: Request },
ctx: TContext,
): Promise<{ result: TResult }>;
): Promise<{ result: TResult; providerInfo?: TProviderInfo }>;
}
/** @public */
export function createProxyAuthenticator<TContext, TResult>(
authenticator: ProxyAuthenticator<TContext, TResult>,
): ProxyAuthenticator<TContext, TResult> {
export function createProxyAuthenticator<TContext, TResult, TProviderInfo>(
authenticator: ProxyAuthenticator<TContext, TResult, TProviderInfo>,
): ProxyAuthenticator<TContext, TResult, TProviderInfo> {
return authenticator;
}