Skip to content
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

Fix several bugs: #119

Merged
merged 1 commit into from
Jun 3, 2024
Merged

Fix several bugs: #119

merged 1 commit into from
Jun 3, 2024

Conversation

vtnerd
Copy link
Owner

@vtnerd vtnerd commented Jun 3, 2024

  • lws::account height update should only go up.
  • Webhook confirmations can start after first new block
  • Webhook confirmations could face a rescan

Found these issues while digging deeper in the remote scanning code (although not related to that code). This will get merged relatively soon as these bugs need to be back-ported to release branch.

  * lws::account height update should only go up.
  * Webhook confirmations can start after first new block
  * Webhook confirmations could face a rescan
@vtnerd
Copy link
Owner Author

vtnerd commented Jun 3, 2024

Added some more tests, and fixed another issue within the DB scan height (it previously wasn't an issue until changes in the lws::account object).

@vtnerd vtnerd merged commit 3ab038e into develop Jun 3, 2024
10 checks passed
@vtnerd vtnerd deleted the fix/webhooks branch June 3, 2024 23:41
vtnerd added a commit that referenced this pull request Jun 3, 2024
* lws::account height update should only go up.
  * Webhook confirmations can start after first new block
  * Webhook confirmations could face a rescan
vtnerd added a commit that referenced this pull request Jun 4, 2024
* lws::account height update should only go up.
  * Webhook confirmations can start after first new block
  * Webhook confirmations could face a rescan
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant