Feat(renderToPipeableStream): Allow passing crossorigin attribute on bootstrapScripts

[HenriqueLimas]

When using window.onerror in the bootstrap script that is hosted in a CDN the error message will show "Script error." instead of a proper error message. This is because of the cross-origin script. To fix this issue a crossorigin="anonymous" attribute needs to be added in the script tag.

Currently, bootstrapScripts supports src and integrity as configurations, should it also supports a crossorigin property?

[ElmoJones3]

+1

[pixelhuh]

Right now, bootstrapScripts lacks flexibility because we can't set scripts attributes. My application needs custom data-attributes on the scripts for compatibility, but I can't set them, this is complicates development.

it would be cool to be able to do something like this:

const { pipe } = renderToPipeableStream(<Page someProp="dummy" />, {
    bootstrapScripts: [{
        src: './main.js',
        crossorigin: true,
        'data-foo': 'bar',
    }],
    onShellReady() {
        res.setHeader('content-type', 'text/html');
        pipe(res);
    },
});

With this result:

<script src="/main.js" async="" crossorigin data-foo="bar"></script>

[rerodrigues]

+1

[gaearon]

Would someone like to send a PR?

[HenriqueLimas]

Sure, let me do that