Skip to content

Conversation

@viral-sangani
Copy link

Description

  • add Celo network addresses for weETH and wstETH tokens

@viral-sangani viral-sangani requested a review from a team as a code owner September 16, 2025 12:52
@wiz-inc-a178a98b5d
Copy link

wiz-inc-a178a98b5d bot commented Sep 16, 2025

Wiz Scan Summary

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

View scan details in Wiz

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

@fainashalts
Copy link
Contributor

Hi @viral-sangani this PR is failing validation, looks like there's an issue with the bridge information. May need nobridge or a bridge override set. Thanks!

@its-everdred
Copy link

Hi @viral-sangani bumping this once more, please review the validation failures.

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*9C...**a0 (42 chars)
ethereum.address 0x*fE...**ee (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).


To get more details on how to remediate this issue using AI, reply to this conversation with #wiz remediate

@viral-sangani
Copy link
Author

Hey @its-everdred sorry for the late reply. I have made the changes but looks like validations are still failing because of Optimism's values.
https://app.circleci.com/pipelines/github/ethereum-optimism/ethereum-optimism.github.io/2033/workflows/6888bf82-522f-497c-80a9-7e20dad424ba/jobs/5629/artifacts

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.

3 participants