doc: mention case-insensitive env on windows by oliversalzburg · Pull Request #9166 · nodejs/node · GitHub

Checklist

make -j8 test (UNIX), or vcbuild test nosign (Windows) passes
tests and/or benchmarks are included
documentation is changed or added
commit message follows commit guidelines

Affected core subsystem(s)

doc

Description of change

On Windows OS, environment variables are case-insensitive and are treated likewise in NodeJS. This can be confusing and can lead to hard-to-debug problems when moving code from one environment to another.

Closes #9157

Building Node fails on my system, so I'm currently unable to run the tests :(

This is the output I get during the build: https://gist.github.com/oliversalzburg/592885b45dbf0a561147faef519b0b05

Building Node fails on my system, so I'm currently unable to run the tests :(

You can just use your system node to run this particular test, like node test/parallel/test-process-env.js.

I think our linter doesn’t like files that don’t end in newlines

Done

I think you’ll need to change require('../common'); to const common = require('../common'); for this to work

Done

@addaleax My system node says it's fine:

$ node test/parallel/test-process-env.js; echo $?
0

…unless I'm not doing it right.

CI: https://ci.nodejs.org/job/node-test-commit/5666/

Can you please capitalize and punctuate the comments.

Gladly, I just wanted to conform with other comments in the file. Is there some guideline regarding capitalization here?

Can you replace this with two separate assertions:

assert.strictEqual(process.env.TEST, 'test');
assert.strictEqual(process.env.teST, 'test');

Done

Instead of checking the typeof, can you just directly compare against undefined.

Done

Actually, you can move the test for process.env.TEST out of the if.

I updated the assertions. Please let me know if that's in line with what you had in mind.

This is the output I get during the build: https://gist.github.com/oliversalzburg/592885b45dbf0a561147faef519b0b05

Yeah that seems like something for @nodejs/platform-windows to look at

This is the output I get during the build: https://gist.github.com/oliversalzburg/592885b45dbf0a561147faef519b0b05

Since it doesn't pertain to the PR, I suggest moving this discussion to #node-dev.

Tested master, it compiles fine. I would suggest git clean -fdx

Nit: can you please swap the arguments? The first one is actual and second one expected.

Good call! Done.

Minor nit: Remove the Note that, and just make it On Windows, environment variables are case-insensitive.

Should be fixed now. Thanks

On Windows OS, environment variables are case-insensitive and are treated likewise in NodeJS. This can be confusing and can lead to hard-to-debug problems when moving code from one environment to another. Fixes: #9157

Environment variables should be treated case-insensitive on Windows platforms and case-sensitive on UNIX platforms

https://ci.nodejs.org/job/node-test-pull-request/4658/

If no one objects I'll land this if green.

FreeBSD failure unrelated (investigate flaky test-http-server-consumed-timeout on FreeBSD #9227)
Fanned arm build failure only in one case and unrelated (https://ci.nodejs.org/job/node-test-binary-arm/4476/RUN_SUBSET=addons,label=pi1-raspbian-wheezy/console) and is pretty weird to be honest it'd be great if someone from @nodejs/build could take a look.

Landed in 2a45616 and 0a8d0ea

Cheers.

On Windows OS, environment variables are case-insensitive and are treated likewise in NodeJS. This can be confusing and can lead to hard-to-debug problems when moving code from one environment to another. Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> PR-URL: #9166 Fixes: #9157

Environment variables should be treated case-insensitive on Windows platforms and case-sensitive on UNIX platforms. This commit ensures this behavior persists. Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> PR-URL: #9166 Fixes: #9157

On Windows OS, environment variables are case-insensitive and are treated likewise in NodeJS. This can be confusing and can lead to hard-to-debug problems when moving code from one environment to another. Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> PR-URL: #9166 Fixes: #9157

Environment variables should be treated case-insensitive on Windows platforms and case-sensitive on UNIX platforms. This commit ensures this behavior persists. Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> PR-URL: #9166 Fixes: #9157

On Windows OS, environment variables are case-insensitive and are treated likewise in NodeJS. This can be confusing and can lead to hard-to-debug problems when moving code from one environment to another. Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> PR-URL: #9166 Fixes: #9157

Environment variables should be treated case-insensitive on Windows platforms and case-sensitive on UNIX platforms. This commit ensures this behavior persists. Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> PR-URL: #9166 Fixes: #9157

On Windows OS, environment variables are case-insensitive and are treated likewise in NodeJS. This can be confusing and can lead to hard-to-debug problems when moving code from one environment to another. Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> PR-URL: #9166 Fixes: #9157

nodejs-github-bot added doc Issues and PRs related to the documentations. process Issues and PRs related to the process subsystem. labels Oct 18, 2016

rvagg force-pushed the master branch from c133999 to 83c7a88 Compare October 18, 2016 17:02

addaleax reviewed Oct 18, 2016

View reviewed changes

addaleax added the test Issues and PRs related to the tests. label Oct 18, 2016

cjihrig reviewed Oct 18, 2016

View reviewed changes

cjihrig approved these changes Oct 18, 2016

View reviewed changes

lpinca reviewed Oct 24, 2016

View reviewed changes

lpinca approved these changes Oct 24, 2016

View reviewed changes

jasnell reviewed Oct 24, 2016

View reviewed changes

jasnell approved these changes Oct 24, 2016

View reviewed changes

oliversalzburg added 2 commits October 24, 2016 16:15

doc: mention case-insensitive env on windows

a1857ac

On Windows OS, environment variables are case-insensitive and are treated likewise in NodeJS. This can be confusing and can lead to hard-to-debug problems when moving code from one environment to another. Fixes: #9157

test: case sensitivity of env variables

b707743

Environment variables should be treated case-insensitive on Windows platforms and case-sensitive on UNIX platforms

benjamingr closed this Oct 25, 2016

oliversalzburg deleted the feature/env-case branch October 25, 2016 09:28

evanlucas mentioned this pull request Nov 3, 2016

v7.1.0 proposal - 2016-11-08 #9438

Merged

MylesBorins added dont-land-on-v4.x labels Nov 17, 2016

MylesBorins mentioned this pull request Nov 22, 2016

v6.9.2 proposal #9735

Merged

coderabbitai Bot mentioned this pull request Apr 5, 2026

Master DevMadhup/Wanderlust-Mega-Project#6

Open

coderabbitai Bot mentioned this pull request May 6, 2026

#17 Add multi-account claude with copy-on-rotate session resume ralphkrauss/agent-orchestrator#46

Merged

6 tasks

doc: mention case-insensitive env on windows by oliversalzburg · Pull Request #9166 · nodejs/node · GitHub

Uh oh!

Uh oh!

doc: mention case-insensitive env on windows - #9166

doc: mention case-insensitive env on windows#9166oliversalzburg wants to merge 2 commits intonodejs:masternodejs/node:masterfrom oliversalzburg:feature/env-caseCopy head branch name to clipboard

Conversation

oliversalzburg commented Oct 18, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Checklist

Affected core subsystem(s)

Description of change

Uh oh!

oliversalzburg commented Oct 18, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

addaleax left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

oliversalzburg commented Oct 18, 2016

Uh oh!

addaleax commented Oct 18, 2016

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

addaleax commented Oct 18, 2016

Uh oh!

seishun commented Oct 19, 2016

Uh oh!

bzoz commented Oct 19, 2016

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

benjamingr commented Oct 25, 2016

Uh oh!

benjamingr commented Oct 25, 2016

Uh oh!

benjamingr commented Oct 25, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

10 participants

doc: mention case-insensitive env on windows#9166
oliversalzburg wants to merge 2 commits into
nodejs:masterfrom
oliversalzburg:feature/env-case

oliversalzburg commented Oct 18, 2016 •
edited
Loading

oliversalzburg commented Oct 18, 2016 •
edited
Loading