-
Notifications
You must be signed in to change notification settings - Fork 158
Add JSON-LD verification via BTC public key #442
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
Conversation
e4551ad
to
9bdc26a
Compare
@ChristopherA @msporny @aquabu @gkellogg this is ready to merge IMO. Please take a look at the demo: http://thawing-inlet-70039.herokuapp.com/playground/ and/or the code, and let's give it some 👍 or 👎 . Thanks! |
Works for me, fine with it getting merged. Pulling in @dlongley and @davidlehn for comments. I have the following requests for future discussion:
Only the third item really needs work before we suggest the signature format for broader use. The first two is on us to make sure it's all more friendly to use in the DV Playground. Thanks a ton for all the hard work @harlantwood. +1 from me to merge. |
9bdc26a
to
b8d6b95
Compare
Yeah, this one was weird, so I pushed a fix. |
I'm not sure if it's quite what you mean, but if you paste in a different (valid) public key ( eg try I clarified the verbiage a bit in the UI:
The latest code is at http://thawing-inlet-70039.herokuapp.com/playground/ |
@msporny can we merge this? Any feedback from others? @dlongley @davidlehn @ChristopherA |
Merged to avoid any further hold up. We can make any additional adjustments as needed via new PRs. |
Thanks for merging, @dlongley. Sorry, @harlantwood we never meant this PR to hang out there that long.
I was raising a general concern wrt. digital signatures and the JSON-LD playground, not with anything you did. What I meant was that "A person can't show up with a digitally signed message and just dump it into the JSON-LD input box and then know whether or not the signature is valid". We have no generalized signature verification mechanism. From what I gather, anything dumped into the JSON-LD input box is then re-signed with the value in "Bitcoin (ECDSA Koblitz) Private Key for Signing" and then checked against the public key in "Bitcoin (ECDSA Koblitz) Public Key for Verification". ... and that we just need to all remember to put this very obvious feature in the yet-to-be-created Digital Verification Playground. :) |
Ah makes sense. NP, thanks for feedback and merging. |
We have signature validation with bitcoin keys working and live on a demo server.
We should wait for the following to be merged before merging this PR:
Once https://web-payments.org/contexts/security-v1.jsonld contains EcdsaKoblitzSignature2016 we can:
Nice to have but probably not critical:
web-payments/web-payments.org#41
cc @ChristopherA @msporny