Skip to content

Refund relayer if parachain head has not been updated#1528

Merged
svyatonik merged 3 commits into
masterfrom
refund-for-non-updated-parachain-head
Aug 1, 2022
Merged

Refund relayer if parachain head has not been updated#1528
svyatonik merged 3 commits into
masterfrom
refund-for-non-updated-parachain-head

Conversation

@svyatonik
Copy link
Copy Markdown
Contributor

closes #1440

I was going to use different approach for that initially, but the logic either becomes too complex (parachain head may be rejected at a different levels) or it is almost impossible to do that (e.g. when we have failed to read parachain head from the proof - shall we even try to refund?). So refunding is done similarly to pruning here - we're only refunding if proof is correct, but someone else has already provided better head.

@svyatonik svyatonik added P-Runtime PR-audit-needed A PR has to be audited before going live. labels Aug 1, 2022
@svyatonik svyatonik enabled auto-merge (squash) August 1, 2022 05:45
@svyatonik svyatonik merged commit 7907de4 into master Aug 1, 2022
@svyatonik svyatonik deleted the refund-for-non-updated-parachain-head branch August 1, 2022 06:07
Copy link
Copy Markdown
Collaborator

@serban300 serban300 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good ! Marking the PR as reviewed.

serban300 pushed a commit to serban300/parity-bridges-common that referenced this pull request Mar 27, 2024
serban300 pushed a commit to serban300/parity-bridges-common that referenced this pull request Apr 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P-Runtime PR-audit-needed A PR has to be audited before going live.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refund relayer if parachain head has not been updated

2 participants