Skip to content

ics20 transfers result in deposit events with incorrect assets #1496

@SuperFluffy

Description

@SuperFluffy

ics20 transfers received by sequencer result in deposit events of assets that are prepended with a <dest_port>/<dest_channel>/ prefix even if the received asset is a source.

Instead, if sequencer is the source of the asset, <source_port>/<source_channel>/ should be stripped.

Problematic code:

let trace_with_dest =
prepend_denom_if_not_refund(&denom_trace, dest_port, dest_channel, is_refund);
// check if this is a transfer to a bridge account and
// execute relevant state changes if it is
execute_ics20_transfer_bridge_lock(
state,
recipient,
&trace_with_dest,
packet_amount,
packet_data.memo.clone(),
is_refund,

┆Issue Number: ENG-814

Metadata

Metadata

Assignees

No one assigned

    Labels

    ibcpertaining to ibc, including ics20sequencerpertaining to the astria-sequencer crate

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions