3.x using Object.defineProperty with process.env failed #2998

Activity

1. pmq20

   added a commit that references this issue on Sep 22, 2015

   v8: Object.defineProperty work with process.env

   ...

   This issue will close once commit 76db621 is merged into the 'master' branch.

   76db621
2. brendanashworth

   added

   processIssues and PRs related to the process subsystem.Issues and PRs related to the process subsystem.

   on Sep 22, 2015
3. evanlucas commented on Feb 22, 2016

   

   evanlucas

   on Feb 22, 2016

   Contributor

   More actions

   Closing as we are not planning to do another io.js release. (#3465)
4. evanlucas

   closed this as completedon Feb 22, 2016
5. gucong3000 commented on Feb 6, 2018

   

   gucong3000

   on Feb 6, 2018

   More actions

   setter is not work for me.

   Object.defineProperty(process.env, 'SHELL', {
   	get: function () {
   		console.log('get');
   		return '';
   	},
   	set: function (value) {
   		console.log('set');
   	},
   	enumerable: true,
   });
   console.log(process.env.SHELL);
   process.env.SHELL = 'test';

   E:\work\git-bash-shell (os -> origin) (git-bash-shell@1.3.0)
   $ node -v
   v9.2.0
   
   E:\work\git-bash-shell (os -> origin) (git-bash-shell@1.3.0)
   $ node test.js
   get
6. gucong3000 commented on Feb 7, 2018

   

   gucong3000

   on Feb 7, 2018

   More actions

   I found the right way.

   Object.defineProperties(
   	Object.getPrototypeOf(process.env),
   	env
   );

   👍React with 👍1Reacted by PatrickJS