Skip to content

Add optional boundaries to getregistrationreceipt #97

@sr-gi

Description

@sr-gi
Member

#83 added, amongst other commands, getregistrationreceipt which pulls the latest registration receipt from the wathctower-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.

Activity

sr-gi

sr-gi commented on Feb 6, 2023

@sr-gi
MemberAuthor

@1010adigupta this may be a good issue to start with

1010adigupta

1010adigupta commented on Feb 6, 2023

@1010adigupta

Okay, will start looking into it immediately.

amitpanwar789

amitpanwar789 commented on Mar 2, 2023

@amitpanwar789

@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?

sr-gi

sr-gi commented on Mar 2, 2023

@sr-gi
MemberAuthor

The schema has to remain the same, the query needs to add a boundary (start /end) so every receipt within that boundary is returned.

amitpanwar789

amitpanwar789 commented on Mar 2, 2023

@amitpanwar789

@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?

sr-gi

sr-gi commented on Mar 2, 2023

@sr-gi
MemberAuthor

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.

ShubhamBhut

ShubhamBhut commented on Mar 3, 2023

@ShubhamBhut

I think I forked worng repo. @sr-gi anyway to fix this now ?

sr-gi

sr-gi commented on Mar 3, 2023

@sr-gi
MemberAuthor

You have to create a PR against this repo and most likely rebase your base commit

added a commit that references this issue on Mar 9, 2023
added a commit that references this issue on Mar 30, 2023
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

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      Participants

      @sr-gi@amitpanwar789@ShubhamBhut@1010adigupta

      Issue actions

        Add optional boundaries to `getregistrationreceipt` · Issue #97 · talaia-labs/rust-teos