-
Notifications
You must be signed in to change notification settings - Fork 110
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
RFQ cleanup #1124
RFQ cleanup #1124
Conversation
We only want a single place where SCIDs are calculated from the RFQ ID. This is especially important due to a future change required where lnd will only accept SCIDs in a certain range and we'll need to deterministically shift the derived SCID to be within that range.
Eliminating these error types ensures that all current errors align with those documented in the BLIP.
Pull Request Test Coverage Report for Build 10887430285Details
💛 - Coveralls |
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
|
||
scidInteger := binary.BigEndian.Uint64(scidBytes) | ||
return SerialisedScid(scidInteger) | ||
return q.ID.Scid() |
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.
note for commit message:
This code should be relevant for generating RFQ scids within range: lnd/aliasmgr/aliasmgr.go
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.
Yeah, I have that code in a WIP branch (lnd-0-19-staging-rebased
already:
taproot-assets/rfqmsg/messages.go
Line 41 in 97be885
minBlock := uint32(aliasmgr.AliasStartBlockHeight) |
|
||
scidInteger := binary.BigEndian.Uint64(scidBytes) | ||
return SerialisedScid(scidInteger) | ||
return q.ID.Scid() |
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.
Yeah, I have that code in a WIP branch (lnd-0-19-staging-rebased
already:
taproot-assets/rfqmsg/messages.go
Line 41 in 97be885
minBlock := uint32(aliasmgr.AliasStartBlockHeight) |
Non-breaking cleanup commits for the RFQ packages.