Skip to content

Conversation

@legobeat
Copy link
Contributor

@legobeat legobeat commented May 13, 2024

Description

  • Upgrade from obsolete eth-rpc-errors to @metamask/rpc-errors
    • This introduce handling of error causes

See here for some context.

Open in GitHub Codespaces

Related issues

Blocked by

Blocking

Manual testing steps

Screenshots/Recordings

Before

After

Pre-merge author checklist

  • I’ve followed MetaMask Coding Standards.
  • I've completed the PR template to the best of my ability
  • I’ve included tests if applicable
  • I’ve documented my code using JSDoc format if applicable
  • I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

@legobeat legobeat added the DO-NOT-MERGE Pull requests that should not be merged label May 13, 2024
@github-actions
Copy link
Contributor

github-actions bot commented May 13, 2024

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@legobeat legobeat force-pushed the deps-eth-rpc-errors-testing2 branch 3 times, most recently from 81de478 to d2b1465 Compare May 13, 2024 04:50
@legobeat

This comment was marked as resolved.

@legobeat legobeat force-pushed the deps-eth-rpc-errors-testing2 branch 4 times, most recently from 1681c02 to 7dc9514 Compare May 13, 2024 06:57
@legobeat legobeat changed the title testing rpc-errors cause handling fix: testing rpc-errors cause handling May 13, 2024
@legobeat legobeat force-pushed the deps-eth-rpc-errors-testing2 branch 18 times, most recently from 2faf92d to 449fd01 Compare May 13, 2024 21:40
@legobeat legobeat requested review from a team and andreahaku October 11, 2024 08:16
@legobeat legobeat force-pushed the deps-eth-rpc-errors-testing2 branch 2 times, most recently from 7afcd4e to f2a2a9f Compare October 14, 2024 03:58
@legobeat legobeat force-pushed the deps-eth-rpc-errors-testing2 branch from f2a2a9f to 71f6a0a Compare October 14, 2024 20:01
@legobeat legobeat force-pushed the deps-eth-rpc-errors-testing2 branch 2 times, most recently from a6ef3a2 to ca8cbcb Compare October 15, 2024 22:48
@legobeat legobeat force-pushed the deps-eth-rpc-errors-testing2 branch from ca8cbcb to f48cf6f Compare October 16, 2024 20:26
fix error message handling

fix: prefer cause message as error message

lint:fix

fix: update error handling for nft ownership check

fix: handle error details in error cause

fix test

fixup cause handling

Revert "fix test"

This reverts commit e5ae79be58cf22b873ede78ac8edac08832bcaac.

fix test

fix: isErrorWithMessage return type

Refactor: Handling error.cause; add pony-cause

use getErrorMessage for logging errors

WIP: deps: @metamask/rpc-errors@^6.2.1->6.3.0

test: fix testcase

fix: consider cause instead of data.cause when parsing caught error messages

chore: clean up unused cause-handling code in error.ts

chore: update lavamoat policies

revert stringification of generic objects in logErrorWithMessage

lint:fix
MetaMask/rpc-errors#158

- Update test according to new @metamask/rpc-errors behavior

This partially reverts commit aeca6d797fc16d13bcd16f8649159769207ede78.
@Gudahtt Gudahtt force-pushed the deps-eth-rpc-errors-testing2 branch from f48cf6f to 2733a31 Compare October 16, 2024 22:47
@Gudahtt Gudahtt merged commit 70e2c08 into MetaMask:develop Oct 16, 2024
75 of 77 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Oct 16, 2024
@Gudahtt
Copy link
Member

Gudahtt commented Oct 16, 2024

Force-merged to override all-jobs-passed required status check, which is failing due to our SonarCloud checknot yet supporting forks.

@gauthierpetetin gauthierpetetin added release-12.7.0 Issue or pull request that will be included in release 12.7.0 release-12.6.0 Issue or pull request that will be included in release 12.6.0 and removed release-12.7.0 Issue or pull request that will be included in release 12.7.0 labels Oct 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

dependencies Pull requests that update a dependency file release-12.6.0 Issue or pull request that will be included in release 12.6.0 team-application-security Application security team team-confirmations Push issues to confirmations team team-core-extension-ux Core Extension UX team

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

8 participants