[chronik] make the maximum history length for electrum RPCs an init param
ClosedPublic
Actions

Authored by PiRK on Thu, Feb 20, 19:21.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABCb760220009df: [chronik] make the maximum history length for electrum RPCs an init param

Summary

This will allow server operators to decide what the reasonable limit should be, and potentially increase it if they are not worried about DDos attacks.

Test Plan

ninja check-functional

Diff Detail

Repository

rABC Bitcoin ABC

Branch

chronikelectrum_max_history

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 32430
Build 64349: Build Diff	chronik-client-integration-tests · ecash-lib-integration-tests · ecash-agora-integration-tests · lint-circular-dependencies · build-without-wallet · build-diff · build-debug · build-clang-tidy · build-clang · build-chronik-plugins · build-chronik
Build 64348: arc lint + arc unit

Event Timeline

PiRK created this revision.Thu, Feb 20, 19:21

Herald added a reviewer: Restricted Project. · View Herald TranscriptThu, Feb 20, 19:21

PiRK requested review of this revision.Thu, Feb 20, 19:21

PiRK retitled this revision from [chronik] make the maximum history length an init param to [chronik] make the maximum history length for electrum RPCs an init param.Thu, Feb 20, 19:36

Harbormaster completed remote builds in B32430: Diff 52691.Thu, Feb 20, 19:46

Fabien accepted this revision.Thu, Feb 20, 19:46

This revision is now accepted and ready to land.Thu, Feb 20, 19:46

Closed by commit rABCb760220009df: [chronik] make the maximum history length for electrum RPCs an init param (authored by PiRK). · Explain WhyFri, Feb 21, 10:35

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABCb760220009df: [chronik] make the maximum history length for electrum RPCs an init param.