-
Notifications
You must be signed in to change notification settings - Fork 37
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
jellyfish-crypto
: aes256
#331
jellyfish-crypto
: aes256
#331
Conversation
Code Climate has analyzed commit d74cf26 and detected 0 issues on this pull request. View more on Code Climate. |
✔️ Deploy Preview for jellyfish-defi ready! 🔨 Explore the source changes: 555d36d 🔍 Inspect the deploy log: https://app.netlify.com/sites/jellyfish-defi/deploys/60b72853fc65a60007a99289 😎 Browse the preview: https://deploy-preview-331--jellyfish-defi.netlify.app |
size-limit report 📦
|
Codecov Report
@@ Coverage Diff @@
## main #331 +/- ##
==========================================
+ Coverage 96.92% 97.15% +0.23%
==========================================
Files 89 92 +3
Lines 2404 2670 +266
Branches 308 342 +34
==========================================
+ Hits 2330 2594 +264
- Misses 74 76 +2
Continue to review full report at Codecov.
|
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.
1 more test for safety
…rs behind (#331) * reconfigure probes for idb defid.readiness dead if defid blocks are more than 4 headers behind model.readiness dead synced blocks are more than 4 blocks behind * added probes healthy with peers testing * Update _model.probes.ts
…rs behind (#331) * reconfigure probes for idb defid.readiness dead if defid blocks are more than 4 headers behind model.readiness dead synced blocks are more than 4 blocks behind * added probes healthy with peers testing * Update _model.probes.ts
What kind of PR is this?:
/kind feature
What this PR does / why we need it:
Dependency of #133 .
Existing lib in found in npm has no TS @types support, so build a simple one directly from
crypto
.Simplified implementation, all arg and return are buffer only.
Which issue(s) does this PR fixes?:
Fixes #
Additional comments?: