-
Notifications
You must be signed in to change notification settings - Fork 339
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
Replace OUR_MAX_HTLCS
with config knob
#2138
Replace OUR_MAX_HTLCS
with config knob
#2138
Conversation
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.
You should add a config knob in ChannelHandshakeConfig
in config.rs
.
Codecov ReportPatch coverage:
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more Additional details and impacted files@@ Coverage Diff @@
## main #2138 +/- ##
==========================================
- Coverage 91.35% 91.33% -0.03%
==========================================
Files 102 102
Lines 50382 50389 +7
Branches 50382 50389 +7
==========================================
- Hits 46027 46021 -6
- Misses 4355 4368 +13
... and 3 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
Still needs more docs on the knob ( #2138 (comment) ) but the code looks good to me! When you do address that, feel free to rebase and squash so that its a clean set of non-merge commits based on upstream (see https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md#rebasing-changes for how to). |
Probably can mark this non-draft too, when you get there :) |
54ca4fa
to
09b9a19
Compare
09b9a19
to
f8deb84
Compare
OUR_MAX_HTLCS
with config knob
c99da4c
to
44ff074
Compare
44ff074
to
fce8013
Compare
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.
LGTM! We'll get another reviewer on this Monday, thanks!
holder_max_accepted_htlcs. Set upper bound of 483 Writes an even TLV if the value isn't 50
fce8013
to
f656609
Compare
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.
looks good!
Add a config knob for
holder_max_accepted_htlcs
to replace the constantOUR_MAX_HTLCS
.resolves #1990