chore: keep responseRedirectURI in AuthorizationRequestState

Author: sanderPostma

packages/siop-oid4vp/lib/rp/InMemoryRPSessionManager.ts

@@ -199,6 +199,7 @@ export class InMemoryRPSessionManager implements IRPSessionManager {
         queryId: event.queryId ?? this.queryIdMapping[event.correlationId],
         ...(type === 'request' && { request: event.subject }),
         ...(type === 'response' && { response: event.subject }),
+        ...(type === 'request' && event.responseRedirectURI && {responseRedirectURI: event.responseRedirectURI}),
         ...(event.error && { error: event.error }),
         status,
         timestamp: event.timestamp,

packages/siop-oid4vp/lib/rp/RP.ts

@@ -93,7 +93,8 @@ export class RP {
     version?: SupportedVersion
     requestByReferenceURI?: string
     responseURI?: string
-    responseURIType?: ResponseURIType
+    responseURIType?: ResponseURIType,
+    responseRedirectURI?: string
   }): Promise<AuthorizationRequest> {
     const authorizationRequestOpts = this.newAuthorizationRequestOpts(opts)
 
@@ -131,7 +132,8 @@ export class RP {
     requestByReferenceURI?: string
     responseURI?: string
     responseURIType?: ResponseURIType
-    callback?: CallbackOpts
+    callback?: CallbackOpts,
+    responseRedirectURI?: string
   }): Promise<URI> {
     const authorizationRequestOpts = this.newAuthorizationRequestOpts(opts)
 
@@ -147,7 +149,8 @@ export class RP {
         correlationId: opts.correlationId,
         queryId: opts.queryId,
         subject: authRequest,
-        callback: opts.callback
+        callback: opts.callback,
+        responseRedirectURI: opts.responseRedirectURI
       })
       return uri
     } catch (error) {
@@ -458,6 +461,7 @@ export class RP {
       queryId?: string
       subject?: AuthorizationRequest | AuthorizationResponse | AuthorizationResponsePayload
       callback?: CallbackOpts
+      responseRedirectURI?: string
       error?: Error
     },
   ): void {

packages/siop-oid4vp/lib/types/Events.ts

@@ -33,14 +33,23 @@ export class AuthorizationEvent<T> {
   private readonly _timestamp: number
   private readonly _correlationId: string
   private readonly _queryId: string
-
-  public constructor(args: { correlationId: string; queryId?: string, subject?: T; callback?: CallbackOpts, error?: Error }) {
+  private readonly _responseRedirectURI: string
+
+  public constructor(args: {
+    correlationId: string;
+    queryId?: string,
+    subject?: T;
+    callback?: CallbackOpts;
+    responseRedirectURI?: string,
+    error?: Error
+  }) {
     //fixme: Create correlationId if not provided. Might need to be deferred to registry though
     this._correlationId = args.correlationId
     this._queryId = args.queryId
     this._timestamp = Date.now()
     this._subject = args.subject
     this._callback = args.callback
+    this._responseRedirectURI = args.responseRedirectURI
     this._error = args.error
   }
 
@@ -60,6 +69,10 @@ export class AuthorizationEvent<T> {
     return this._callback
   }
 
+  get responseRedirectURI(): string {
+    return this._responseRedirectURI
+  }
+
   public hasError(): boolean {
     return !!this._error
   }

packages/siop-oid4vp/lib/types/SessionManager.ts

@@ -8,6 +8,7 @@ export interface AuthorizationRequestState {
   request: AuthorizationRequest
   status: AuthorizationRequestStateStatus
   callback?: CallbackOpts
+  responseRedirectURI?: string
   timestamp: number
   lastUpdated: number
   error?: Error