Skip to content

Add Bridged EURC to Lisk #1157

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

ricott1
Copy link
Contributor

@ricott1 ricott1 commented Jun 2, 2025

Description

Bridged EURC is deployed on Lisk using Bridged EURC Standard for the OP Stack which also deploys and uses new L1 and L2 bridges. The deployment is adapted from the Bridged USDC deployment (just renaming USDC to EURC).

  • Added Bridged EURC deployment address, name and symbol on Lisk
  • Added L1 bridge address for Lisk on Ethereum
  • Added L2 bridge address on Lisk

Tests

npx tsx ./bin/cli.ts validate --datadir ./data --tokens BridgedEURC

@ricott1 ricott1 requested review from wbnns and a team as code owners June 2, 2025 15:10
Copy link

wiz-inc-a178a98b5d bot commented Jun 2, 2025

Wiz Scan Summary

Scanner Findings
Vulnerability Finding Vulnerabilities
Data Finding Sensitive Data 1 Low
Secret Finding Secrets
IaC Misconfiguration IaC Misconfigurations
Total 1 Low

View scan details in Wiz

To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension.

@@ -0,0 +1,24 @@
{

Choose a reason for hiding this comment

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

Low Sensitive Data Finding

Financial

More Details
Attribute Value
Data Classifier Financial/Ethereum Address
Data Classifier ID BUILTIN-388

Sampled Examples

Key Value
ethereum/address 0x*aE...**3c (42 chars)

Rule ID: BUILTIN-388


To ignore this finding as an exception, reply to this conversation with #wiz_ignore reason

If you'd like to ignore this finding in all future scans, add an exception in the .wiz file (learn more) or create an Ignore Rule (learn more).

Copy link
Collaborator

Choose a reason for hiding this comment

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

#wiz_ignore the contract address is expected

Choose a reason for hiding this comment

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

The finding has been successfully ignored for these checks.

If you'd like to ignore this finding in all future scans, you can add a matching exception in the .wiz file.
Learn more

@shuse2 shuse2 self-requested a review June 2, 2025 15:13
@shuse2 shuse2 added the lisk PR require review from lisk team label Jun 2, 2025
@shuse2
Copy link
Collaborator

shuse2 commented Jun 4, 2025

@wbnns it's all good from our side. Could you take a look?

@shuse2 shuse2 requested review from fainashalts and removed request for fainashalts June 12, 2025 06:37
@shuse2
Copy link
Collaborator

shuse2 commented Jun 12, 2025

@fainashalts could you take a look at the PR if you are available ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lisk PR require review from lisk team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants