René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Issue about `os.homedir()`","articleBody":"I saw `os.homedir()` is directly using `uv_os_homedir` and it has an strange feature.\n\n\u003e See [here](https://github.com/libuv/libuv/blob/v1.x/src/unix/core.c#L1016) for the feature.\n\nIt will check the environment variable first and then check for the real home directory.\n\nAnd someone discussed [here](https://github.com/libuv/libuv/pull/350#issuecomment-102701904) for the reason.\n\n\u003e I wonder if this should check $HOME first? It's something of a UNIX tradition to be able to change your home directory on the fly. A problem with that is that getenv(\"HOME\") is not MT-safe.\n\u003e \n\u003e -- @bnoordhuis\n\nBut I think there's still a problem.\n\n---\n\nConsider one situation:\n\n\u003e If I wrote a CLI program and it will write some log file on user's home directory.\n\u003e \n\u003e Now I run the CLI like this:\n\u003e \n\u003e ``` sh\n\u003e $ sudo -u foo node mycli.js\n\u003e ```\n\u003e \n\u003e It should write log file on `/home/foo`. But the problem is - `os.homedir()` still got `process.env.HOME`.\n\nOne solution is to edit `/etc/sudoers`:\n\n```\n# env_keep+=\"HOME MAIL\"\n# comment env_keep += \"HOME\"\n```\n\nAnd another method is using `-i` argument.\n\n**_For further reading you can go to [this article](http://f10.moe/2016/03/07/%E4%B8%80%E4%B8%AA%E7%94%B1%E4%BA%8E%E7%8E%AF%E5%A2%83%E5%8F%98%E9%87%8F%E4%BA%A7%E7%94%9F%E7%9A%84bug/). (It was written in Chinese by @Amunu)**_\n\n---\n\n**But I think why don't we wrote a `os.realHomedir()` or removing checking `HOME` in `realhomedir()`? Not all of us (normal developers) have the permission to access configuration files on our online servers.**\n","author":{"url":"https://github.com/XadillaX","@type":"Person","name":"XadillaX"},"datePublished":"2016-03-07T08:27:41.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":8},"url":"https://github.com/5582/node/issues/5582"}