Skip to content

Update @metamask/inpage-provider from v8.0.3 to v8.0.4#10378

Merged
kumavis merged 1 commit intodevelopfrom
update-inpage-provider
Feb 8, 2021
Merged

Update @metamask/inpage-provider from v8.0.3 to v8.0.4#10378
kumavis merged 1 commit intodevelopfrom
update-inpage-provider

Conversation

@Gudahtt
Copy link
Copy Markdown
Member

@Gudahtt Gudahtt commented Feb 5, 2021

Fixes #10356

There was a bug in the inpage provider that would mistakenly report usage of our injected web3 instance when the web3.currentProvider property was accessed. This was fixed in v8.0.4 of @metamask/inpage-provider.

Manual testing steps:

  • Access the currentProvider property of our injected web3 shim multiple times
  • See that an error is not shown in the console, and that a warning is not shown in the MetaMask extension popup UI

Fixes #10356

There was a bug in the inpage provider that would mistakenly report
usage of our injected `web3` instance when the `web3.currentProvider`
property was accessed. This was fixed in v8.0.4 of
`@metamask/inpage-provider`.
@Gudahtt Gudahtt requested a review from a team as a code owner February 5, 2021 00:01
@Gudahtt Gudahtt requested a review from danjm February 5, 2021 00:01
@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [fff8016]
Page Load Metrics (606 ± 30 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint478460115
domContentLoaded4127306056230
load4137316066230
domInteractive4117296046230

@kumavis kumavis merged commit 9dc8839 into develop Feb 8, 2021
@kumavis kumavis deleted the update-inpage-provider branch February 8, 2021 12:41
@github-actions github-actions bot locked and limited conversation to collaborators Feb 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Removed web3 API notification trigger?

4 participants