Skip to content

chore: upgrade hardhat and gas reporter deps, disable gas reporting #26

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

Closed
wants to merge 2 commits into from

Conversation

jeffsmale90
Copy link
Collaborator

@jeffsmale90 jeffsmale90 commented Jul 7, 2025

Upgrades hardhat and gas reporter dependencies, in order to resolve the following High priority vulnerability https://github.com/MetaMask/delegation-toolkit/security/dependabot/15

Bumps resolution of pbkdf2 (in the hardhat dependency chain) to resolve critical vulnerability https://github.com/MetaMask/delegation-toolkit/security/dependabot/25

@jeffsmale90 jeffsmale90 requested a review from a team as a code owner July 7, 2025 23:50
Copy link

socket-security bot commented Jul 7, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedhardhat-gas-reporter@​1.0.10 ⏵ 2.3.098 -1100100 +184 +3100

View full report

Copy link

socket-security bot commented Jul 7, 2025

Caution

Review the following alerts detected in dependencies.

According to your organization's Security Policy, you must resolve all "Block" alerts before proceeding. Learn more about Socket for GitHub.

Action Severity Alert (click for details)
Block Medium
[email protected] has Network access.

Module: globalThis["fetch"]

Location: Package overview

From: ?npm/[email protected]npm/[email protected]

ℹ Read more on: This package | This alert | What is network access?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at [email protected].

Suggestion: Packages should remove all network access that is functionally unnecessary. Consumers should audit network access to ensure legitimate use.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/[email protected]. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

@jeffsmale90 jeffsmale90 marked this pull request as draft July 8, 2025 01:56
@jeffsmale90 jeffsmale90 closed this Jul 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant