fix: Create request options were not merged properly with existing op…

…tions
Sphereon-Opensource · Jan 10, 2024 · 1f8815b · 1f8815b
1 parent 926d480
commit 1f8815b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/rp/RP.ts b/src/rp/RP.ts
@@ -49,8 +49,7 @@ export class RP {
     verifyResponseOpts?: VerifyAuthorizationResponseOpts;
   }) {
     // const claims = opts.builder?.claims || opts.createRequestOpts?.payload.claims;
-    const authReqOpts = createRequestOptsFromBuilderOrExistingOpts(opts);
-    this._createRequestOptions = { ...authReqOpts, payload: { ...authReqOpts.payload } };
+    this._createRequestOptions = createRequestOptsFromBuilderOrExistingOpts(opts);
     this._verifyResponseOptions = { ...createVerifyResponseOptsFromBuilderOrExistingOpts(opts) };
     this._eventEmitter = opts.builder?.eventEmitter;
     this._sessionManager = opts.builder?.sessionManager;

diff --git a/src/rp/RPBuilder.ts b/src/rp/RPBuilder.ts
@@ -236,11 +236,12 @@ export class RPBuilder {
 
   withPresentationDefinition(definitionOpts: { definition: IPresentationDefinition; definitionUri?: string }, targets?: PropertyTargets): RPBuilder {
     const { definition, definitionUri } = definitionOpts;
-    const definitionProperties = {
-      presentation_definition: definition,
-      presentation_definition_uri: definitionUri,
-    };
+
     if (this.getSupportedRequestVersion() < SupportedVersion.SIOPv2_D11) {
+      const definitionProperties = {
+        presentation_definition: definition,
+        presentation_definition_uri: definitionUri,
+      };
       const vp_token = { ...definitionProperties };
       if (isTarget(PropertyTarget.AUTHORIZATION_REQUEST, targets)) {
         this._authorizationRequestPayload.claims = {