Skip to content

Commit

Permalink
BOTL 7: fixup: clarify when to reply with a node_announcement
Browse files Browse the repository at this point in the history
Do not reply with a node_announcement if the query includes an optional query flag that does not request it.
The current wording could be interpreted as "always follow with node announcements whenever
you reply with a channel announcements" which defeats the point of using query flags (if you want the node
announcements just set the corresponding bits).
  • Loading branch information
sstone committed Sep 16, 2019
1 parent 313c0f2 commit c8e53fe
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions 07-routing-gossip.md
Original file line number Diff line number Diff line change
Expand Up @@ -660,6 +660,7 @@ The receiver:
- MUST respond to each known `short_channel_id`:
- if the incoming message does not include `encoded_query_flags`:
- with a `channel_announcement` and the latest `channel_update` for each end
- MUST follow with any `node_announcement`s for each `channel_announcement`
- otherwise:
- We define `query_flag` for the Nth `short_channel_id` in
`encoded_short_ids` to be the Nth varint of the decoded
Expand All @@ -675,8 +676,7 @@ The receiver:
- if bit 4 of `query_flag` is set and it has received a `node_announcement` from `node_id_2`:
- MUST reply with the latest `node_announcement` for `node_id_2`
- SHOULD NOT wait for the next outgoing gossip flush to send these.
- MUST follow with any `node_announcement`s for each `channel_announcement`
- SHOULD avoid sending duplicate `node_announcements` in response to a single `query_short_channel_ids`.
- SHOULD avoid sending duplicate `node_announcements` in response to a single `query_short_channel_ids`.
- MUST follow these responses with `reply_short_channel_ids_end`.
- if does not maintain up-to-date channel information for `chain_hash`:
- MUST set `complete` to 0.
Expand Down

0 comments on commit c8e53fe

Please sign in to comment.