Environment variable evaluation not working in index.html

[marco-gagliardi]

Is this a bug report?

yes

Did you try recovering your dependencies?

yes

Environment

System:
OS: macOS High Sierra 10.13.6
CPU: x64 Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz
Binaries:
Node: 10.9.0 - /usr/local/bin/node
npm: 6.2.0 - /usr/local/bin/npm
Browsers:
Chrome: 70.0.3538.67
Safari: 12.0
npmPackages:
react: ^16.5.2 => 16.5.2
react-dom: ^16.5.2 => 16.5.2
react-scripts: ^2.0.3 => 2.0.5
npmGlobalPackages:
create-react-app: Not Found

Steps to Reproduce

1. Add a condition script in your public/index.html file based on an env variable:

<script>
      '%REACT_APP_ENV%' === 'production' ?  console.log('production') : console.log('not production')
</script>

2. Set a value for that env variable at build time: REACT_APP_ENV=production react-scripts build
3. Inspect build/index.html

Expected Behavior

You should see

<script>console.log('production')</script>

Actual Behavior

you see:

<script>console.log('not production')</script>

[jtwebb]

I can't give a satisfying reason as to why this is happening, but I can give a fix.

<script>
      '%REACT_APP_ENV%'.toString() === 'production' ?  console.log('production') : console.log('not production')
</script>

I'm not sure why we would have to add toString() because if you check the types on both '%REACT_APP_ENV%' and 'production' they're both strings. And the values are both 'production'. But as soon as I added the toString() it worked as expected.

[marco-gagliardi]

Maybe they are not at build time, when the condition is evaluated. Thanks for the temporary fix though!

[Poyoman39]

+1 Have the same issue.

        console.log('%REACT_APP_CONFIGURATION_BEHAVIOR%');
        console.log('web');
        console.log('%REACT_APP_CONFIGURATION_BEHAVIOR%' == 'web');

got => web web false

It looks like the environements variables have differents values / are not set inside a build tool (probably webpack). Adding ".toString()" probably delay the evaluation of the expression at runtime

Note: I'm using "react-scripts": "2.1.1",

[stale]

This issue has been automatically marked as stale because it has not had any recent activity. It will be closed in 5 days if no further activity occurs.

[stale]

This issue has been automatically closed because it has not had any recent activity. If you have a question or comment, please open a new issue.

[andreisocaciu]

console.log('%REACT_APP_LOAD_STUFF%' === 'true') still returns false.