Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invalid Frame on verification of a BBS signature #122

Open
vongohren opened this issue May 6, 2021 · 8 comments
Open

Invalid Frame on verification of a BBS signature #122

vongohren opened this issue May 6, 2021 · 8 comments

Comments

@vongohren
Copy link

Im uncertain where this fails but seems to me that it comes from this lib.

I have a strane error coming from trying to recreate the bbs signature sample with libraries that I want to use.
I try to great a keypair with did-key.js and use that to sign.
That works
Then I want to verify but get a strange JSON LD verification error, and dont know where to start.

Any thoughts?

Screenshot 2021-05-06 at 17 03 15

Screenshot 2021-05-06 at 17 03 30

@vongohren
Copy link
Author

Some context

Screen Shot 2021-05-06 at 19 15 10

@kdenhartog
Copy link
Contributor

Can you publish the code that you're using to work with this? I'm thinking the issue here is that the verificationMethod used is only a fragment which means either an @base property needs to be specified within the issued credential or the full did url syntax should be added so that the documentLoader can resolve the public key material to verify the signatures.

@vongohren
Copy link
Author

vongohren commented May 6, 2021

Thanks for reply @kdenhartog. Anywhere I can read about this @base property, or you know about any examples or helpers about that? There is btw a long thread going on here: https://difdn.slack.com/archives/C4X50SNUX/p1620313732057300, as it was relevant for some other questions I had up in the air.

I will see if your pointer and the discussion leads to something working tomorrow, dont have the code at hand now.

Will close tomorrow if successful

@vongohren
Copy link
Author

@kdenhartog I do think I touched upon something bigger: w3c/did-imp-guide#3, not directly related but there are some mentions of BBS and documents without context there.

@kdenhartog kdenhartog self-assigned this May 6, 2021
@vongohren
Copy link
Author

I was not able to get past this issue, even if I change up things in the document loader: https://gist.github.com/vongohren/92753c7dc8e8be7ed219c2d184c624aa

How shall I be able to verify this?

@vongohren
Copy link
Author

I did get help from another DIF member: https://gist.github.com/S3bb1/ddd74704c0f9980d93e4e6302439594c
He left comments and thoughts on why it was needed.

But is this the stuff you have to deal with usually? Or since everyone is making their own things, they know their own libraries?

@tplooker
Copy link
Member

@vongohren to be clear is this issue distinct from #114?

@vongohren
Copy link
Author

@tplooker yeah thsi is different, as this is more related to the resolving. The other I believe is mainly related to the big gap in different versions for Json-ld signatures.

@kdenhartog kdenhartog removed their assignment May 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants