-
Notifications
You must be signed in to change notification settings - Fork 0
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
metafeed/announce is signed by meta feed #45
Conversation
Oh, I should probably update validation to check these sigs. |
Oh well actually So this PR is ready to go I guess |
Or hey I can just make a new function, |
@@ -123,7 +123,7 @@ test('seed', (t) => { | |||
}) | |||
|
|||
test('announce', (t) => { | |||
messages.getContentAnnounce(metafeedKeys.id, (err, content) => { | |||
messages.getContentAnnounce(metafeedKeys, (err, content) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There must have been a bug here before?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes there was. The checking is done correctly in test/messages.js
but in this file we want to test the query APIs so we just used getContentAnnounce
to create the inputs, but we didn't validate the input. Anyway whatever :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just found that minor spelling thing, otherwise looks good
For issue ssbc/ssb-meta-feeds-spec#27