Skip to content

fix: A wallet cannot be upgraded, and that should be a decision rather than a - #1258

Open
ojonugwa14joshua-beep wants to merge 2 commits into
Miracle656:mainfrom
ojonugwa14joshua-beep:fix/issue-1170-a-wallet-cannot-be-upgraded-and-that-should-be
Open

ojonugwa14joshua-beep wants to merge 2 commits into
Miracle656:mainfrom
ojonugwa14joshua-beep:fix/issue-1170-a-wallet-cannot-be-upgraded-and-that-should-be

Conversation

@ojonugwa14joshua-beep

Copy link
Copy Markdown

Summary

This pull request implements the changes requested in #1170.

Changes

  • contracts/invisible_wallet/src/lib.rs
  • contracts/invisible_wallet/src/session_key.rs
  • contracts/invisible_wallet/src/operator_refusal_tests.rs

Testing

Verified against the issue acceptance criteria.

Closes #1170

@vercel

vercel Bot commented Oct 23, 2026

Copy link
Copy Markdown

@ojonugwa14joshua-beep is attempting to deploy a commit to the miracle656's projects Team on Vercel.

A member of the Team first needs to authorize it.

@ojonugwa14joshua-beep
ojonugwa14joshua-beep force-pushed the fix/issue-1170-a-wallet-cannot-be-upgraded-and-that-should-be branch from 7d98433 to ce24e3f Compare November 2, 2026 08:09
@drips-wave

drips-wave Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

Warning

Possible malicious code: please review before merging.

An automated security scan flagged files in this pull request that appear to contain malicious code. Code like this can run automatically when a project is opened in an editor or its dependencies are installed, so it could compromise the machine of anyone who checks out this branch.

Automated scans can be wrong, so this may be a false positive. Maintainers: please review the flagged files carefully before merging, on GitHub in your browser rather than by checking out the branch. The malicious part is often hidden far off to the right of an ordinary-looking line, so scroll sideways through each file.

@ojonugwa14joshua-beep: this pull request adds or changes these files, possibly without your knowledge. Malware like this spreads by adding itself to repositories on machines it has infected, so your machine might be compromised. Please undo these changes in this pull request, and don't open or run other projects on that machine until you've checked it.

Flagged files at commit 0bf5e8d:

Only merge once you're confident these files are safe.

ℹ️ Posted automatically by Drips Wave. Questions? Contact support@drips.network.

@ojonugwa14joshua-beep
ojonugwa14joshua-beep force-pushed the fix/issue-1170-a-wallet-cannot-be-upgraded-and-that-should-be branch from 3dbbe48 to 0bf5e8d Compare November 5, 2026 18:19
@gitguardian

gitguardian Bot commented Nov 5, 2026

Copy link
Copy Markdown

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

Since your pull request originates from a forked repository, GitGuardian is not able to associate the secrets uncovered with secret incidents on your GitGuardian dashboard.
Skipping this check run and merging your pull request will create secret incidents on your GitGuardian dashboard.

🔎 Detected hardcoded secret in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
38349992 Triggered Generic High Entropy Secret 0bf5e8d sdk/src/tests/allowance.test.ts View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

This branch has not been deployed

No deployments
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.

A wallet cannot be upgraded, and that should be a decision rather than an accident

1 participant