Skip to content
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

1.x: Fix EIP-1193 provider subscriptions #3864

Merged
merged 1 commit into from Jan 20, 2021

Conversation

rekmarks
Copy link

@rekmarks rekmarks commented Jan 19, 2021

Fixes #3651

Implements subscription support per EIP-1193.

Should be tested manually by maintainers with current MetaMask (^9.0.0) on Firefox.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 496709829

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 399 unchanged lines in 8 files lost coverage.
  • Overall coverage decreased (-2.6%) to 73.808%

Files with Coverage Reduction New Missed Lines %
packages/web3-core-requestmanager/src/jsonrpc.js 1 88.0%
packages/web3-core-requestmanager/lib/index.js 14 83.7%
packages/web3-core-helpers/src/formatters.js 24 79.65%
packages/web3-core-helpers/src/errors.js 31 4.41%
packages/web3-utils/src/soliditySha3.js 55 5.56%
packages/web3-utils/src/index.js 60 30.0%
packages/web3-utils/src/utils.js 84 13.24%
packages/web3-eth-accounts/src/index.js 130 31.65%
Totals Coverage Status
Change from base Build 496406655: -2.6%
Covered Lines: 3287
Relevant Lines: 4217

💛 - Coveralls

@frankiebee
Copy link
Contributor

Cool thanks for this will take a look at this!

@rekmarks
Copy link
Author

FYI: I'm receiving reports that using patch-package with this PR fixes the issue.

Copy link
Contributor

@GregTheGreek GregTheGreek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@spacesailor24 spacesailor24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Installed this patch locally and tested using Firefox and MetaMask v9.0.1

Copy link
Contributor

@frankiebee frankiebee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

purely from a code stand point this looks good!

@GregTheGreek GregTheGreek merged commit bf70c0f into web3:1.x Jan 20, 2021
GregTheGreek added a commit that referenced this pull request Jan 20, 2021
1.x: Fix EIP-1193 provider subscriptions
@rekmarks rekmarks deleted the fix-1.x-subscriptions branch January 20, 2021 20:55
GregTheGreek added a commit that referenced this pull request Jan 21, 2021
1.x: Fix EIP-1193 provider subscriptions
r-czajkowski added a commit to keep-network/keep-core that referenced this pull request Jan 22, 2021
This v1.3.2 version is a hot fix to address breaking changes in the
MetaMask API provided in v.9.
Details:
web3/web3.js#3864
MetaMask/metamask-extension#8077 (comment)
r-czajkowski added a commit to keep-network/tbtc-dapp that referenced this pull request Jan 22, 2021
This v1.3.2 version is a hot fix to address breaking changes in the
MetaMask API provided in v.9.
Details:
web3/web3.js#3864
MetaMask/metamask-extension#8077 (comment)
r-czajkowski added a commit to keep-network/keep-core that referenced this pull request Jan 25, 2021
This v1.3.3 version is a hot fix to address breaking changes in the
MetaMask API provided in v.9.
Details:
web3/web3.js#3864
MetaMask/metamask-extension#8077 (comment)
r-czajkowski added a commit to keep-network/tbtc-dapp that referenced this pull request Jan 25, 2021
This v1.3.3 version is a hot fix to address breaking changes in the
MetaMask API provided in v.9.
Details:
web3/web3.js#3864
MetaMask/metamask-extension#8077 (comment)
michalsmiarowski added a commit to keep-network/keep-core that referenced this pull request Jan 27, 2021
…bscription

Bump web3 dependency to v1.3.3

This v1.3.3 version is a hot fix to address breaking changes in the
MetaMask API provided in v.9.
Details:
web3/web3.js#3864
MetaMask/metamask-extension#8077 (comment)
r-czajkowski added a commit to keep-network/tbtc-dapp that referenced this pull request May 12, 2021
This v1.3.3 version is a hot fix to address breaking changes in the
MetaMask API provided in v.9.
Details:
web3/web3.js#3864
MetaMask/metamask-extension#8077 (comment)
r-czajkowski added a commit to keep-network/tbtc-dapp that referenced this pull request May 12, 2021
Bump web3 dependencies to the latest version v1.3.5 to address breaking
changes in the MetaMask API provided in v.9.
Details:
web3/web3.js#3864
MetaMask/metamask-extension#8077 (comment)
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.

RequestManager depends on deprecated event when used with MetaMask
5 participants