Mention case-insensitivity in process.env docs

[oliversalzburg]

On Windows OS, environment variables are case-insensitive.

14:46 $ node
> process.env.foo = "lol"
'lol'
> process.env.fOo
'lol'

I think it could be beneficial to make a note about this in the docs.

[addaleax]

It might also be good to add tests for this behaviour to detect platform differences or keep us from accidentally changing anything here.

Would you be interested in writing a pull request to add this yourself? The files to edit should be doc/api/process.md and/or test/parallel/test-process-env.js.

[oliversalzburg]

@addaleax I'm not quite sure how to properly test this behavior, as the behavior would likely change depending on which OS you're running the test on. If you have any suggestions, please let me know and I'd be happy to write the test as well.

[addaleax]

@oliversalzburg There’s https://github.com/nodejs/node/blob/master/doc/guides/writing_tests.md for generic “How to write a test” things, and you can check for common.isWindows in tests to see whether the test is running on Windows or not. Ideally, you’d check that other OS show the opposite behaviour, to make sure that it’s really only Windows that does this.

[oliversalzburg]

@addaleax Ah, sounds good. Thanks for the help :)

[oliversalzburg]

@addaleax Is there also a guide for building Node on Windows? Because I'm getting a bunch of linker errors when trying to vcbuild test nosign with Visual Studio 2015.

[addaleax]

I am afraid I can only ask @nodejs/platform-windows, but it might help if you have the errors somewhere in a gist or something

[oliversalzburg]

Alright, I'll put the information into the PR. Thanks again

[joaocgreis]

For reference, guide for building on Windows: https://github.com/nodejs/node/blob/master/BUILDING.md#windows

[nfischer]

So does this mean that process.env is guaranteed to have case-insensitive keys on Windows, even going back to v4? I've hunted in the past for documentation, couldn't find anything, so I just assumed that was the case. Thanks.