Skip to content

Add methods to manage channel fee policies and retrieve channels by outpoint#504

Open
markettes wants to merge 12 commits into
mainfrom
feat/set-channel-fee-policy
Open

Add methods to manage channel fee policies and retrieve channels by outpoint#504
markettes wants to merge 12 commits into
mainfrom
feat/set-channel-fee-policy

Conversation

@markettes
Copy link
Copy Markdown
Contributor

Introduce methods for setting channel fee policies and retrieving channels by their outpoint. Implement unit tests to ensure functionality and correctness of these new features.

@markettes markettes requested a review from Jossec101 May 6, 2026 08:15
Comment thread src/Proto/nodeguard.proto
message AddTagsResponse {
}

message SetChannelFeePolicyRequest {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pls doc each field?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

throw new ArgumentException("Both inboundBaseFeeMsat and inboundFeeRatePpm must be provided together for inbound fee policy.");
}

var channel = await _channelRepository.GetByOutpoint(outPoint);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Another validation is that inboundFeeRatePpm needs to be <= normal fee ppm rate. Im not sure if LND validates this but we can do also for defensive programming

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@markettes markettes force-pushed the feat/set-channel-fee-policy branch from cdc1049 to bf09174 Compare May 13, 2026 08:30
@markettes markettes force-pushed the feat/set-channel-fee-policy branch from d64426b to a9e00a8 Compare May 13, 2026 14:56
@markettes markettes requested a review from Jossec101 May 14, 2026 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants