-
Notifications
You must be signed in to change notification settings - Fork 64
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
Add optional boundaries to getregistrationreceipt
#97
Comments
@1010adigupta this may be a good issue to start with |
Okay, will start looking into it immediately. |
@sr-gi can you clarify whether we need to change the schema to add a timestamp column or get a registration receipt on the basis of subscription_start and end? |
The schema has to remain the same, the query needs to add a boundary (start /end) so every receipt within that boundary is returned. |
@sr-gi I will like to work on it, so what should be the format of our optional parameters, I guess we need to filter out it from DB on the basis of subscription_expiry, which is I guess the blocks count till which tower must cover the client. optional parameter can be blocks count. am i thinking in right direction? |
The boundary should be a start and end block, given subscription times are accounted for in blocks. https://github.com/talaia-labs/rust-teos/blob/master/watchtower-plugin/src/dbm.rs#L53-L54 This should be as straight-forward as adding two additional params to the RPC command, if those are present (both of them) then the DB query must be bounded by them, otherwise we pull the most recent receipt. |
I think I forked worng repo. @sr-gi anyway to fix this now ? |
You have to create a PR against this repo and most likely rebase your base commit |
#83 added, amongst other commands,
getregistrationreceipt
which pulls the latest registration receipt from thewathctower-plugin
database.It'll be nice to add some optional parameters that will allow the user to pick the receipt they are interested in, in case it is not the lattest. The parameters should specify a start and end time, and the receipt (or receipts) that englobe such period should be returned.
The text was updated successfully, but these errors were encountered: