Repository navigation
Update wisp to v0.7.0 - #21
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
📒 Files selected for processing (2)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. WalkthroughThe Docker build now pins Wisp v0.7.0 and its specified commit. The current StartOS version and five localized release notes also change to v0.7.0. ChangesWisp v0.7.0 Upgrade
Priority: ⬆️ High Estimated code review effort: 2 (Simple) | ~10 minutes Change: Other · Severity of issue fixed: High Suggested reviewers: Merge Risk: ⚪ Minimal · up to The package is pinned to the verified v0.7.0 commit, and the inspected settings schema is unchanged. No concrete merge-blocking issue was established. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 2 systems. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. A rabbit checks the version pin, Comment |
Bumps the pinned upstream release from v0.6.0 to v0.7.0 (commit f5f3dbb). v0.6.1 was never packaged, so its connection lifecycle fixes come along.
v0.7.0 is a security release: an AUTH event signed for another relay could be altered to authenticate as that user, and an out-of-range filter value or malformed sync message could crash the relay (this package builds ReleaseSafe, where it aborted the process) or pin a CPU core. It also brings multi-filter and
limit: 0handling in line with NIP-01, makes NIP-78 app data private to its author, adds the new NIP-86 methods, and applies bans, allowlists and limits to events the spider syncs. No settings change and no migration.Built the x86_64 image from this Dockerfile and ran it: it starts as v0.7.0, NIP-11 reports
0.7.0with NIP-78 advertised, and a REQ with"limit": 99999999999gets EOSE instead of aborting.Summary by CodeRabbit