Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@yihuang
Copy link
Collaborator

@yihuang yihuang commented Oct 27, 2025

Description

Closes: #770


Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

  • tackled an existing issue or discussed with a team member
  • left instructions on how to review the changes
  • targeted the main branch

Signed-off-by: yihuang <[email protected]>
@yihuang yihuang changed the title Problem: ibc middleware verify sender address incorrectly fix: ibc middleware verify sender address incorrectly Oct 27, 2025
@yihuang yihuang requested a review from vladjdk October 28, 2025 15:38
@aljo242
Copy link
Contributor

aljo242 commented Oct 28, 2025

@yihuang lint failing

Copy link
Member

@vladjdk vladjdk 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 now, let's just fix the lint

@yihuang
Copy link
Collaborator Author

yihuang commented Oct 29, 2025

looks good now, let's just fix the lint

lint and unit tests fixed.

@aljo242
Copy link
Contributor

aljo242 commented Oct 30, 2025

@Mergifyio backport release/v0.5.x

@aljo242 aljo242 added this pull request to the merge queue Oct 30, 2025
@mergify
Copy link

mergify bot commented Oct 30, 2025

backport release/v0.5.x

⚠️ Cannot use the command backport release/v0.5.x

Details

⚠ The product Workflow Automation needs to be activated to enable this feature.

@aljo242
Copy link
Contributor

aljo242 commented Oct 30, 2025

@Mergifyio backport release/v0.5.x

@mergify
Copy link

mergify bot commented Oct 30, 2025

backport release/v0.5.x

✅ Backports have been created

Details

Merged via the queue into cosmos:main with commit 91519e4 Oct 30, 2025
20 checks passed
mergify bot pushed a commit that referenced this pull request Oct 30, 2025
* Problem: ibc middleware verify sender address incorrectly

* Update CHANGELOG.md

Signed-off-by: yihuang <[email protected]>

* remove sendToSelf check

* fix test build

* fix test

* Fix test

* fix test

---------

Signed-off-by: yihuang <[email protected]>
Co-authored-by: Vlad J <[email protected]>
Co-authored-by: Alex | Cosmos Labs <[email protected]>
(cherry picked from commit 91519e4)

# Conflicts:
#	CHANGELOG.md
aljo242 added a commit that referenced this pull request Oct 30, 2025
…#782)

* fix: ibc middleware verify sender address incorrectly (#769)

* Problem: ibc middleware verify sender address incorrectly

* Update CHANGELOG.md

Signed-off-by: yihuang <[email protected]>

* remove sendToSelf check

* fix test build

* fix test

* Fix test

* fix test

---------

Signed-off-by: yihuang <[email protected]>
Co-authored-by: Vlad J <[email protected]>
Co-authored-by: Alex | Cosmos Labs <[email protected]>
(cherry picked from commit 91519e4)

# Conflicts:
#	CHANGELOG.md

* fix

---------

Co-authored-by: yihuang <[email protected]>
Co-authored-by: aljo242 <[email protected]>
@yihuang yihuang deleted the fix-sender-address branch November 1, 2025 00:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Problem: erc20 ibc middleware parse sender incorrectly

3 participants