Add support for script nonce attributes

[dustinsoftware]

Fixes #430, #482, #334

[dustinsoftware]

@Daniel15 this is good to go, but I'd like another set of eyes to give it a quick look before it gets merged 😄

[dustinsoftware]

Merging this so it doesn't get stale, let me know if needs modification before shipping. Tested on Mvc4 and CoreMvc.

[Daniel15]

Thanks. Sorry I never got to this, I think you sent it while I was on vacation.

[Daniel15]

Isn't there already a getter for this? ReactSiteConfiguration.Configuration or .Current or something like that?

[dustinsoftware]

I can check later, the prop already existed on the implementation but not the interface.. don’t remember seeing another one I could use

…

On Sun, Mar 4, 2018 at 18:59, Daniel Lo Nigro ***@***.***> wrote: ***@***.**** commented on this pull request. Thanks. Sorry I never got to this, I think you sent it while I was on vacation. ------------------------------ In src/React.Core/IReactEnvironment.cs <#496 (comment)>: > @@ -110,5 +109,10 @@ public interface IReactEnvironment /// Returns the currently held JS engine to the pool. (no-op if engine pooling is disabled) /// </summary> void ReturnEngineToPool(); + + /// <summary> + /// Gets the site-wide configuration. + /// </summary> + IReactSiteConfiguration Configuration { get; } Isn't there already a getter for this? ReactSiteConfiguration.Configuration or .Current or something like that? — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#496 (review)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AA5hFhWbsxbP6a5XAkSahhQRgTXl1l20ks5tbKnSgaJpZM4RdZap> .