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

feat: handle for packet timeout #86

Merged
merged 21 commits into from
Jun 28, 2023
Merged

feat: handle for packet timeout #86

merged 21 commits into from
Jun 28, 2023

Conversation

viveksharmapoudel
Copy link
Collaborator

No description provided.

@viveksharmapoudel viveksharmapoudel marked this pull request as draft June 21, 2023 12:43
@codecov
Copy link

codecov bot commented Jun 21, 2023

Codecov Report

Merging #86 (23efe32) into main (54f31ed) will decrease coverage by 0.09%.
The diff coverage is 0.50%.

❗ Current head 23efe32 differs from pull request most recent head 3cf1f6c. Consider uploading reports for the commit 3cf1f6c to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #86      +/-   ##
==========================================
- Coverage   11.34%   11.25%   -0.09%     
==========================================
  Files          82       82              
  Lines       18593    18695     +102     
==========================================
- Hits         2109     2105       -4     
- Misses      16255    16361     +106     
  Partials      229      229              
Impacted Files Coverage Δ
relayer/chains/archway/event_parser.go 0.00% <ø> (ø)
relayer/chains/archway/helper_debug_msg.go 0.00% <0.00%> (ø)
relayer/chains/archway/msg.go 0.00% <ø> (ø)
relayer/chains/archway/provider.go 19.38% <0.00%> (-0.14%) ⬇️
relayer/chains/archway/query.go 0.00% <0.00%> (ø)
relayer/chains/archway/tx.go 0.00% <0.00%> (ø)
relayer/chains/archway/utils.go 50.00% <0.00%> (-6.25%) ⬇️
relayer/chains/cosmos/tx.go 0.00% <0.00%> (ø)
relayer/chains/icon/event_parser.go 22.55% <0.00%> (ø)
relayer/chains/icon/events.go 47.50% <ø> (ø)
... and 11 more

@viveksharmapoudel viveksharmapoudel marked this pull request as ready for review June 26, 2023 04:17
@izyak izyak linked an issue Jun 28, 2023 that may be closed by this pull request
@izyak izyak self-requested a review June 28, 2023 04:54
@izyak izyak merged commit 2e5ea7c into main Jun 28, 2023
@izyak izyak deleted the feat/packet-timeout branch June 28, 2023 04:56
izyak pushed a commit to izyak/ibc-relay-test that referenced this pull request Sep 7, 2023
* fix: helper method issue on no json data

* fix: archway query refractor

* chore: add helper

* feat: add timeoutpacket and requestTimeoutpacket method

* feat: add packet timeout events on icon

* feat: adding icon block for each height not only btp blocks

* feat: proof on queryPacketCommitment

* feat: add icon requesttimeout and packet timeout method

* feat: add Msgtimeoutpacket and request type

* fix: handler shouldsendpacket check condition

* feat:handle processor for timeout packet

* fix: packetReceipt in icon

* fix: handle assemble message for request timeout

* fix: validate packet method

* fix: archway validate method

* fix: handle for request timeout proof

* fix: seperate updateclient and other messages in archway (icon-project#87)

* fix: seperate updateclient and other messages in archway

* fix: increase retrySend after 2 block

* fix: change request timeout type and proof function

* fix: packet timeout

* chore: name from common constant
izyak pushed a commit that referenced this pull request Sep 8, 2023
* fix: helper method issue on no json data

* fix: archway query refractor

* chore: add helper

* feat: add timeoutpacket and requestTimeoutpacket method

* feat: add packet timeout events on icon

* feat: adding icon block for each height not only btp blocks

* feat: proof on queryPacketCommitment

* feat: add icon requesttimeout and packet timeout method

* feat: add Msgtimeoutpacket and request type

* fix: handler shouldsendpacket check condition

* feat:handle processor for timeout packet

* fix: packetReceipt in icon

* fix: handle assemble message for request timeout

* fix: validate packet method

* fix: archway validate method

* fix: handle for request timeout proof

* fix: seperate updateclient and other messages in archway (#87)

* fix: seperate updateclient and other messages in archway

* fix: increase retrySend after 2 block

* fix: change request timeout type and proof function

* fix: packet timeout

* chore: name from common constant
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.

Handle Packet Timeout for Unordered Channel
2 participants