deriveProof does not support blank node identifiers for credentialSubject

[OR13]

Originally discovered by @brianorwhatever

See is reproduction here:

transmute-industries/verifiable-data#76

Surfacing from the linked PR description:

Related issues:

• bundle with src? #121
• Removing credentialSubject.id causes null credentialSubject #128

Questions:

1. Is it possible to use deriveProof on a credential with no credentialSubject.id? this example implies that it should be possible...

[OR13]

Current hypothesis is that is may be VC related by not LD Suite related... working to confirm that on the linked PR.

[brianorwhatever]

I have attempted to recreate with just the linked data proof from example 12 above and ran into an issue with the id already being a node identifier

removing the id derived a proof properly however I'm now running into a internal error in Neon module: index out of bounds: the len is 5 but the index is 5 problem on verification

[OR13]

Hmm, this test maybe covers this case https://github.com/mattrglobal/jsonld-signatures-bbs/blob/master/__tests__/AnonymousVC.spec.ts

[tmarkovski]

What's the suggested fix to this issue? Force the use of id in credentialSubject?

[tplooker]

@tmarkovski that would be a suggested interim solution however the long term solution is a revision to the proof suite that accounts for this case in the URDNA algorithm. Given the recent consensus we have had around decentralized-identity/bbs-signature#10 I'm working on an updated implementation that incorporates this among various other changes