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

Bug: Proxy on Context throws an error in DevTools #19545

Closed
timothykrell-hb opened this issue Aug 6, 2020 · 3 comments · Fixed by #19584
Closed

Bug: Proxy on Context throws an error in DevTools #19545

timothykrell-hb opened this issue Aug 6, 2020 · 3 comments · Fixed by #19584

Comments

@timothykrell-hb
Copy link

React version: 16.12.0

Steps To Reproduce

  1. Create an app with a Proxy function that has a get method that returns a function stored in Context
  2. Attempt to look at Provider component in devtools

Link to code example:
https://codesandbox.io/s/stoic-jepsen-7jjfn?file=/src/App.js
https://7jjfn.csb.app/

The current behavior

When using the devtools to inspect the Context.Provider, devtools throw an error:

Uncaught DOMException: Failed to execute 'postMessage' on 'Window': function () {
        return null;
      } could not be cloned.

Components using the context do not show info in devtools and show "Loading..." instead.

The expected behavior

No error is thrown.

Possibly Related

#16691

@timothykrell-hb timothykrell-hb added the Status: Unconfirmed A potential issue that we haven't yet confirmed as a bug label Aug 6, 2020
@timothykrell-hb timothykrell-hb changed the title Bug: Proxy on Context throws an error Bug: Proxy on Context throws an error in DevTools Aug 6, 2020
@bvaughn bvaughn added Component: Developer Tools good first issue Type: Bug and removed Status: Unconfirmed A potential issue that we haven't yet confirmed as a bug labels Aug 6, 2020
@pfongkye
Copy link
Contributor

pfongkye commented Aug 6, 2020

Hello,
I would like to work on this issue. If it's ok, any pointer to where to start will be welcomed.
Thanks.

@omarsy
Copy link
Contributor

omarsy commented Aug 10, 2020

@pfongkye You’re still working on this issue?

@pfongkye
Copy link
Contributor

@pfongkye You’re still working on this issue?

Yes. Just got started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants