Skip to content

[wallets] Update Safe SDK #2221

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

Merged
merged 2 commits into from
Jan 26, 2024
Merged

[wallets] Update Safe SDK #2221

merged 2 commits into from
Jan 26, 2024

Conversation

MananTank
Copy link
Member

No description provided.

@MananTank MananTank requested a review from a team January 25, 2024 22:27
Copy link

changeset-bot bot commented Jan 25, 2024

🦋 Changeset detected

Latest commit: a91552d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 8 packages
Name Type
@thirdweb-dev/wallets Patch
@thirdweb-dev/auth Patch
thirdweb Patch
@thirdweb-dev/react-core Patch
@thirdweb-dev/react-native Patch
@thirdweb-dev/react Patch
@thirdweb-dev/unity-js-bridge Patch
@thirdweb-dev/react-native-compat Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

socket-security bot commented Jan 25, 2024

New dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@safe-global/[email protected] Transitive: environment, eval, filesystem, network, shell, unsafe +336 58.1 MB dasanra

View full report↗︎

Copy link

socket-security bot commented Jan 25, 2024

👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

Ignoring: npm/[email protected], npm/[email protected], npm/[email protected]

View full report↗︎

Next steps

Take a deeper look at the dependency

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 support [AT] socket [DOT] dev.

Remove the package

If you happen to install a dependency that Socket reports as Known Malware you should immediately remove it and select a different dependency. For other alert types, you may may wish to investigate alternative packages or consider if there are other ways to mitigate the specific risk posed by the dependency.

Mark a package as acceptable risk

To ignore an alert, reply with a comment starting with @SocketSecurity ignore followed by a space separated list of ecosystem/package-name@version specifiers. e.g. @SocketSecurity ignore npm/[email protected] or ignore all packages with @SocketSecurity ignore-all

@MananTank MananTank marked this pull request as draft January 25, 2024 22:32
@MananTank MananTank added DO NOT MERGE This pull request is still in progress and is not ready to be merged. and removed DO NOT MERGE This pull request is still in progress and is not ready to be merged. labels Jan 25, 2024
@MananTank MananTank marked this pull request as ready for review January 26, 2024 16:36
@MananTank
Copy link
Member Author

@SocketSecurity ignore npm/[email protected] npm/[email protected] npm/[email protected]

@MananTank MananTank added this pull request to the merge queue Jan 26, 2024
Merged via the queue into main with commit 4d89dc1 Jan 26, 2024
@MananTank MananTank deleted the mnn/safe-update-sdk branch January 26, 2024 18:07
@jnsdls jnsdls mentioned this pull request Jan 26, 2024
jnsdls pushed a commit that referenced this pull request Jun 19, 2024
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