String is modified in the production build

[agentcooper]

Is this a bug report?

Yes.

Did you try recovering your dependencies?

This is about a clean new project.

Environment

1. node -v: 8.9.4
2. npm -v: 5.6.0
3. npm ls react-scripts: 1.1.1
4. Operating system: macOS

Steps to reproduce

1. create-react-app minification-test && cd minification-test
2. Replace App.js with

import React, { Component } from 'react';

class App extends Component {
 render() {
 const test = `<Script></Script>`;

 return (
 <p>{test}</p>
 );
 }
}

export default App;

3. npm run build
4. serve -s build (npm install serve -g if needed)

Expected behavior

I expect to see <Script></Script> on the screen.

Actual behavior

I see <Script></script> – the second script is lowercase.

Reproducible demo

npm start:

Build:

[agentcooper]

This looks like an UglifyJS issue, it is reproducible in UglifyJS playground with following code:

const test = `<Script></Script>`;

and inline_script option enabled.

[Timer]

Could you please file this issue with Uglify and link back here? Thank you!

[unknownMarty]

Same issue here. I'm using react-chartkick that requires charts.js Guess this is related
No charting library found for t Removing Uglify inside webpack.config.prod.js is temp workaround for me.

[Timer]

Should we disable inline_script for now? What are those implications?

Can you please try react-scripts@2 and see if it persists in the beta?

[agentcooper]

Linking the issue: mishoo/UglifyJS#2989.

[miraage]

FYI: mishoo/UglifyJS#2991

[Timer]

Looks like this was fixed in Uglify. It'll work its way in here eventually.

Thanks for the report!