Closed snape479 closed 4 months ago
The fix looks good for me. Thanks @snape479
But I would propose to hold this fix, since we share the same codebase as Grin but Grin probably remove the whole cache in a PR (https://github.com/mimblewimble/grin/pull/3628).
At versions 5.3 the cache is removed form the node at grin and in mwc-node. Closing this PR because of this
This pull request re-enables the verifier_cache with fixes to avoid the inflation bug. It also includes two tests. The 'test_overflow_cached_rangeproof' test can be run against checkins before the verifier_cache was disabled to confirm failure prior to that change.