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

[BUG] Ensure caller substrate and evm address always match #868

Closed
1xstj opened this issue Jan 3, 2025 · 0 comments · Fixed by #872
Closed

[BUG] Ensure caller substrate and evm address always match #868

1xstj opened this issue Jan 3, 2025 · 0 comments · Fixed by #872
Assignees
Labels
p0 🔥 Critical issues that need to be resolved immediately

Comments

@1xstj
Copy link
Contributor

1xstj commented Jan 3, 2025

pub fn deposit(
origin: OriginFor<T>,
asset_id: Asset<T::AssetId>,
amount: BalanceOf<T>,
evm_address: Option<H160>,
) -> DispatchResult {
let who = ensure_signed(origin)?;
Self::process_deposit(who.clone(), asset_id, amount, evm_address)?;
Self::deposit_event(Event::Deposited { who, amount, asset_id });
Ok(())
}

@1xstj 1xstj self-assigned this Jan 3, 2025
@1xstj 1xstj added the p0 🔥 Critical issues that need to be resolved immediately label Jan 3, 2025
@github-project-automation github-project-automation bot moved this to Not Started 🕧 in Webb Universe Jan 3, 2025
@shekohex shekohex assigned shekohex and unassigned 1xstj Jan 6, 2025
shekohex added a commit that referenced this issue Jan 7, 2025
Fixed some calls in the pallets, also this fixes an issue or bug in

Closes #868
shekohex added a commit that referenced this issue Jan 9, 2025
Fixed some calls in the pallets, also this fixes an issue or bug in

Closes #868
@github-project-automation github-project-automation bot moved this from Not Started 🕧 to Completed ✅ in Webb Universe Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p0 🔥 Critical issues that need to be resolved immediately
Projects
Status: Completed ✅
Development

Successfully merging a pull request may close this issue.

2 participants