Proxy get handler is used when evaluating the proxy #16483

Activity

1. kuanee

   changed the title [-]Proxy getter raises on evaluation of proxy[/-] [+]Proxy get handler is used when evaluating the proxy[/+] on Oct 25, 2017
2. bnoordhuis commented on Oct 25, 2017

   

   bnoordhuis

   on Oct 25, 2017

   Member

   More actions

   I think missing from your description is that it happens in the REPL?
3. vsemozhetbyt commented on Oct 25, 2017

   

   vsemozhetbyt

   on Oct 25, 2017

   Last edited by vsemozhetbyt

   Contributor

   More actions

   There are some properties that are requested during REPL evaluation of an Object. You can filter them out:

   'use strict';
   
   const util = require('util');
   
   const o1 = {
     ready: false,
   };
   
   setTimeout(() => {
     o1.ready = true;
   }, 1000000000);
   
   const handler = {
     get(target, propKey, receiver) {
       const filter = ['inspect', util.inspect.custom, Symbol.iterator];
       if (!filter.includes(propKey)) {
         if (target.ready === false) {
           throw new Error('not ready');
         } else {
           return 'ready';
         }
       }
     },
   };
   
   const proxy = new Proxy(o1, handler);
   
   proxy; // does not raise 'not ready'

   👍React with 👍1Reacted by Ryan Zhang
4. vsemozhetbyt

   added

   replIssues and PRs related to the REPL subsystem.Issues and PRs related to the REPL subsystem.

   utilIssues and PRs related to the built-in util module.Issues and PRs related to the built-in util module.

   on Oct 25, 2017
5. bnoordhuis

   added a commit that references this issue on Oct 25, 2017

   repl: show proxies as Proxy objects

   ...

   4da7a85
6. bnoordhuis commented on Oct 25, 2017

   

   bnoordhuis

   on Oct 25, 2017

   Member

   More actions

   #16485 - possible solution.
7. kuanee commented on Oct 25, 2017

   

   kuanee

   on Oct 25, 2017

   Author

   More actions

   yup, encountered this in repl. Thanks alot!
8. bnoordhuis

   added a commit that references this issue on Nov 7, 2017

   repl: show proxies as Proxy objects

   ...

   90a4390
9. bnoordhuis

   closed this as completedin 90a4390on Nov 7, 2017