specify protocol version in getSupportedProfiles

src/iden3comm/handlers/auth.ts

@@ -1,4 +1,4 @@
 import { MediaType, ProtocolVersion } from '../constants';
 import { IProofService } from '../../proof/proof-service';
 import { PROTOCOL_MESSAGE_TYPE } from '../constants';
 
@@ -492,17 +492,7 @@
     if (profile?.length) {
       const supportedMediaTypes: MediaType[] = [];
       for (const acceptProfile of profile) {
-        // 1. check protocol version
-        const { protocolVersion, env } = parseAcceptProfile(acceptProfile);
-        const responseTypeVersion = Number(responseType.split('/').at(-2));
-        if (
-          protocolVersion !== ProtocolVersion.V1 ||
-          (protocolVersion === ProtocolVersion.V1 &&
-            (responseTypeVersion < 1 || responseTypeVersion >= 2))
-        ) {
-          continue;
-        }
-        // 2. check packer support
+        const { env } = parseAcceptProfile(acceptProfile);
         if (this._packerMgr.isProfileSupported(env, acceptProfile)) {
           supportedMediaTypes.push(env);
         }

src/iden3comm/packers/jws.ts

@@ -1,5 +1,10 @@
 import { BasicMessage, IPacker, JWSPackerParams } from '../types';
-import { AcceptJwsAlgorithms, MediaType, SUPPORTED_PUBLIC_KEY_TYPES } from '../constants';
+import {
+  AcceptJwsAlgorithms,
+  MediaType,
+  ProtocolVersion,
+  SUPPORTED_PUBLIC_KEY_TYPES
+} from '../constants';
 import { extractPublicKeyBytes, resolveVerificationMethods } from '../utils/did';
 import { keyPath, KMS } from '../../kms/';
 
@@ -105,12 +110,18 @@ export class JWSPacker implements IPacker {
 
   /** {@inheritDoc IPacker.getSupportedProfiles} */
   getSupportedProfiles(): string[] {
-    return [`env=${this.mediaType()}&alg=${this.getSupportedAlgorithms().join(',')}`];
+    return this.getSupportedProtocolVersions().map(
+      (v) => `${v};env=${this.mediaType()};alg=${this.getSupportedAlgorithms().join(',')}`
+    );
   }
 
   /** {@inheritDoc IPacker.isProfileSupported} */
   isProfileSupported(profile: string) {
-    const { env, circuits, alg } = parseAcceptProfile(profile);
+    const { protocolVersion, env, circuits, alg } = parseAcceptProfile(profile);
+
+    if (!this.getSupportedProtocolVersions().includes(protocolVersion)) {
+      return false;
+    }
     if (env !== this.mediaType()) {
       return false;
     }
@@ -129,6 +140,10 @@ export class JWSPacker implements IPacker {
     return [AcceptJwsAlgorithms.ES256K, AcceptJwsAlgorithms.ES256KR];
   }
 
+  private getSupportedProtocolVersions(): ProtocolVersion[] {
+    return [ProtocolVersion.V1];
+  }
+
   private async resolveDidDoc(from: string) {
     let didDocument: DIDDocument;
     try {

src/iden3comm/packers/plain.ts

@@ -1,5 +1,5 @@
 import { BasicMessage, IPacker } from '../types';
-import { MediaType } from '../constants';
+import { MediaType, ProtocolVersion } from '../constants';
 import { byteDecoder, byteEncoder } from '../../utils';
 import { parseAcceptProfile } from '../utils';
 
@@ -57,12 +57,16 @@ export class PlainPacker implements IPacker {
 
   /** {@inheritDoc IPacker.getSupportedProfiles} */
   getSupportedProfiles(): string[] {
-    return [`env=${this.mediaType()}`];
+    return this.getSupportedProtocolVersions().map((v) => `${v};env=${this.mediaType()}`);
   }
 
   /** {@inheritDoc IPacker.isProfileSupported} */
   isProfileSupported(profile: string) {
-    const { env, circuits, alg } = parseAcceptProfile(profile);
+    const { protocolVersion, env, circuits, alg } = parseAcceptProfile(profile);
+
+    if (!this.getSupportedProtocolVersions().includes(protocolVersion)) {
+      return false;
+    }
     if (env !== this.mediaType()) {
       return false;
     }
@@ -77,4 +81,8 @@ export class PlainPacker implements IPacker {
 
     return true;
   }
+
+  private getSupportedProtocolVersions(): ProtocolVersion[] {
+    return [ProtocolVersion.V1];
+  }
 }

src/iden3comm/packers/zkp.ts

@@ -20,7 +20,7 @@ import {
   ErrStateVerificationFailed,
   ErrUnknownCircuitID
 } from '../errors';
-import { AcceptAuthCircuits, AcceptJwzAlgorithms, MediaType } from '../constants';
+import { AcceptAuthCircuits, AcceptJwzAlgorithms, MediaType, ProtocolVersion } from '../constants';
 import { byteDecoder, byteEncoder } from '../../utils';
 import { DEFAULT_AUTH_VERIFY_DELAY } from '../constants';
 import { parseAcceptProfile } from '../utils';
@@ -178,16 +178,22 @@ export class ZKPPacker implements IPacker {
 
   /** {@inheritDoc IPacker.getSupportedProfiles} */
   getSupportedProfiles(): string[] {
-    return [
-      `env=${this.mediaType()}&alg=${this.getSupportedAlgorithms().join(
-        ','
-      )}&circuitIds=${this.getSupportedCircuitIds().join(',')}`
-    ];
+    return this.getSupportedProtocolVersions().map(
+      (v) =>
+        `${v};env=${this.mediaType()};alg=${this.getSupportedAlgorithms().join(
+          ','
+        )};circuitIds=${this.getSupportedCircuitIds().join(',')}`
+    );
   }
 
   /** {@inheritDoc IPacker.isProfileSupported} */
   isProfileSupported(profile: string) {
-    const { env, circuits, alg } = parseAcceptProfile(profile);
+    const { protocolVersion, env, circuits, alg } = parseAcceptProfile(profile);
+
+    if (!this.getSupportedProtocolVersions().includes(protocolVersion)) {
+      return false;
+    }
+
     if (env !== this.mediaType()) {
       return false;
     }
@@ -209,6 +215,10 @@ export class ZKPPacker implements IPacker {
   private getSupportedCircuitIds(): AcceptAuthCircuits[] {
     return [AcceptAuthCircuits.AuthV2];
   }
+
+  private getSupportedProtocolVersions(): ProtocolVersion[] {
+    return [ProtocolVersion.V1];
+  }
 }
 
 const verifySender = async (token: Token, msg: BasicMessage): Promise<void> => {

tests/handlers/discover-protocol.test.ts

@@ -49,7 +49,7 @@ describe('discovery-protocol', () => {
     expect(disclosures[0][DiscoverFeatureQueryType.FeatureType]).to.be.eq(
       DiscoveryProtocolFeatureType.Accept
     );
-    expect(disclosures[0].id).to.be.eq('env=application/iden3comm-plain-json');
+    expect(disclosures[0].id).to.be.eq('iden3comm/v1;env=application/iden3comm-plain-json');
   });
 
   it('jws and plain message accept disclosures', async () => {
@@ -72,8 +72,10 @@ describe('discovery-protocol', () => {
       DiscoveryProtocolFeatureType.Accept
     );
     const disclosureIds = disclosures.map((d) => d.id);
-    expect(disclosureIds).to.include('env=application/iden3comm-plain-json');
-    expect(disclosureIds).to.include('env=application/iden3comm-signed-json&alg=ES256K,ES256K-R');
+    expect(disclosureIds).to.include('iden3comm/v1;env=application/iden3comm-plain-json');
+    expect(disclosureIds).to.include(
+      'iden3comm/v1;env=application/iden3comm-signed-json;alg=ES256K,ES256K-R'
+    );
   });
 
   it('zkp and plain message accept disclosures', async () => {
@@ -95,9 +97,9 @@ describe('discovery-protocol', () => {
       DiscoveryProtocolFeatureType.Accept
     );
     const disclosureIds = disclosures.map((d) => d.id);
-    expect(disclosureIds).to.include('env=application/iden3comm-plain-json');
+    expect(disclosureIds).to.include('iden3comm/v1;env=application/iden3comm-plain-json');
     expect(disclosureIds).to.include(
-      'env=application/iden3-zkp-json&alg=groth16&circuitIds=authV2'
+      'iden3comm/v1;env=application/iden3-zkp-json;alg=groth16;circuitIds=authV2'
     );
   });
 
@@ -123,11 +125,13 @@ describe('discovery-protocol', () => {
       DiscoveryProtocolFeatureType.Accept
     );
     const disclosureIds = disclosures.map((d) => d.id);
-    expect(disclosureIds).to.include('env=application/iden3comm-plain-json');
+    expect(disclosureIds).to.include('iden3comm/v1;env=application/iden3comm-plain-json');
+    expect(disclosureIds).to.include(
+      'iden3comm/v1;env=application/iden3-zkp-json;alg=groth16;circuitIds=authV2'
+    );
     expect(disclosureIds).to.include(
-      'env=application/iden3-zkp-json&alg=groth16&circuitIds=authV2'
+      'iden3comm/v1;env=application/iden3comm-signed-json;alg=ES256K,ES256K-R'
     );
-    expect(disclosureIds).to.include('env=application/iden3comm-signed-json&alg=ES256K,ES256K-R');
   });
 
   it('zkp, jws and plain message accept disclosures with exact match', async () => {
@@ -140,7 +144,7 @@ describe('discovery-protocol', () => {
     const acceptQueryMessageWithMatch = createDiscoveryFeatureQueryMessage([
       {
         [DiscoverFeatureQueryType.FeatureType]: DiscoveryProtocolFeatureType.Accept,
-        match: 'env=application/iden3-zkp-json&alg=groth16&circuitIds=authV2'
+        match: 'iden3comm/v1;env=application/iden3-zkp-json;alg=groth16;circuitIds=authV2'
       }
     ]);
 
@@ -153,7 +157,7 @@ describe('discovery-protocol', () => {
       DiscoveryProtocolFeatureType.Accept
     );
     expect(disclosures[0].id).to.include(
-      'env=application/iden3-zkp-json&alg=groth16&circuitIds=authV2'
+      'iden3comm/v1;env=application/iden3-zkp-json;alg=groth16;circuitIds=authV2'
     );
   });
 
@@ -306,7 +310,7 @@ describe('discovery-protocol', () => {
       DiscoveryProtocolFeatureType.Protocol
     );
     const disclosureIds = disclosures.map((d) => d.id);
-    expect(disclosureIds).to.include('env=application/iden3comm-plain-json');
+    expect(disclosureIds).to.include('iden3comm/v1;env=application/iden3comm-plain-json');
     expect(disclosureIds).to.include(PROTOCOL_MESSAGE_TYPE.AUTHORIZATION_REQUEST_MESSAGE_TYPE);
     expect(disclosureIds).to.include(PROTOCOL_MESSAGE_TYPE.AUTHORIZATION_REQUEST_MESSAGE_TYPE);
   });

tests/iden3comm/jws.test.ts

@@ -17,6 +17,7 @@ import { DIDResolutionResult } from 'did-resolver';
 import { ES256KSigner } from 'did-jwt';
 import { DID, getChainId, Id } from '@iden3/js-iden3-core';
 import { Hex } from '@iden3/js-crypto';
+import { MediaType } from '../../src/iden3comm/constants';
 
 const didExample = {
   '@context': [
@@ -217,4 +218,9 @@ describe('jws packer tests', () => {
     const data = await packer.unpack(tokenBytes);
     expect(data).to.not.be.undefined;
   });
+
+  it('test getSupportedProfiles', async () => {
+    const [accept] = await packer.getSupportedProfiles();
+    expect(accept).to.be.eq(`iden3comm/v1;env=${MediaType.SignedMessage};alg=ES256K,ES256K-R`);
+  });
 });

tests/iden3comm/packageManager.test.ts

@@ -1,6 +1,7 @@
 import {
   DataPrepareHandlerFunc,
   PackageManager,
+  PlainPacker,
   VerificationHandlerFunc,
   ZKPPacker
 } from '../../src/iden3comm/index';
@@ -70,6 +71,17 @@ describe('tests packageManager with ZKP Packer', () => {
     expect(senderDID.string()).to.deep.equal(unpackedMessage.from);
     expect(byteDecoder.decode(msgBytes)).to.deep.equal(JSON.stringify(unpackedMessage));
   });
+
+  it('test getSupportedProfiles', async () => {
+    const pm = new PackageManager();
+    pm.registerPackers([new ZKPPacker(new Map(), new Map()), new PlainPacker()]);
+    const supportedProfiles = await pm.getSupportedProfiles();
+    expect(supportedProfiles.length).to.be.eq(2);
+    expect(supportedProfiles).to.include(
+      `iden3comm/v1;env=${MediaType.ZKPMessage};alg=groth16;circuitIds=authV2`
+    );
+    expect(supportedProfiles).to.include(`iden3comm/v1;env=${MediaType.PlainMessage}`);
+  });
 });
 
 const createFetchCredentialMessage = (typ: MediaType, from: DID, to: DID) => {

tests/iden3comm/zkp.test.ts

@@ -2,7 +2,7 @@ import { DID } from '@iden3/js-iden3-core';
 import { Token, ProvingMethodAlg } from '@iden3/js-jwz';
 import { ZKPPackerParams } from '../../src/iden3comm/types';
 import { AuthV2PubSignals } from '../../src/circuits';
-import { PROTOCOL_MESSAGE_TYPE } from '../../src/iden3comm/constants';
+import { MediaType, PROTOCOL_MESSAGE_TYPE } from '../../src/iden3comm/constants';
 import { byteDecoder, byteEncoder } from '../../src';
 import { expect } from 'chai';
 import { initZKPPacker } from './mock/proving';
@@ -41,4 +41,12 @@ describe('zkp packer tests', () => {
 
     expect(PROTOCOL_MESSAGE_TYPE.AUTHORIZATION_RESPONSE_MESSAGE_TYPE).to.deep.equal(iden3msg.type);
   });
+
+  it('test getSupportedProfiles', async () => {
+    const p = await initZKPPacker();
+    const [accept] = await p.getSupportedProfiles();
+    expect(accept).to.be.eq(
+      `iden3comm/v1;env=${MediaType.ZKPMessage};alg=groth16;circuitIds=authV2`
+    );
+  });
 });