Skip to content

lnwire: add InboundFee struct in-line within ChannelUpdate #8604

Description

@Roasbeef

In #6703 the new inbound fee field is added. However, it isn't added directly to the struct, instead the caller must always manually copy the field into the opaque data field, and also must always manually parse it out the other end. We should make this new field uniform with the other TLC fields by adding it directly within the struct. While we're at it, we can also morph it into tlv.OptionalRecord.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    P2should be fixed if one has timeenhancementImprovements to existing features / behaviourfeesRelated to the fees paid for transactions (both LN and funding/commitment transactions)inbound feeChanges related to inbound routing feeintermediateIssues suitable for developers moderately familiar with the codebase and LNwire protocolEncoding of messages for the communication between nodes

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions