search

interledger-deprecated / five-bells-shared

Common elements that are shared between Five Bells components
Other
4 stars 5 forks source link

Cleanup unused signing code #138

Closed alandotcom closed 8 years ago

alandotcom commented 8 years ago

Notification signing done using crypto conditions

https://github.com/interledger/five-bells-ledger/pull/235 https://github.com/interledger/five-bells-notary/pull/57

MatthewPhinney commented 8 years ago

LGTM

alandotcom commented 8 years ago

Need to update this first, https://github.com/interledger/five-bells-connector/blob/master/src/models/notifications.js#L4

alandotcom commented 8 years ago

Nevermind, we're still using jsrsasign to sign notifications. https://github.com/interledger/five-bells-ledger/blob/41a438e09eae74e0c87f33b90fdd8b248c0473ad/src/lib/notificationWorker.js#L12