Include prefix.js into Microsoft.ApplicationInsights.JavaScript

[SergeyKanzhelev]

It is a good idea to include the snippet as a script into NuGet package as discussed here:

microsoft/ApplicationInsights-aspnetcore#120 (comment)

[RehanSaeed]

The snippet.html file does not seem to be checked-in, although it is referred to in the NuGet.JavaScript.csproj project as ..\JavaScriptSDK\min\snippet_vsix.html. The same can be said for the ai.js and ai.min.js files. Am I missing something?

In fact the JavaScriptSDK.csproj project does not seem to load either.

[SergeyKanzhelev]

See https://github.com/Microsoft/ApplicationInsights-JS/blob/master/JavaScript/JavaScriptSDK/min/snippet.ps1 referred into JavaScriptSDK project file

[RehanSaeed]

I see. I'm not much of an expert in PowerShell. I could copy the snippet.ps1 file and create one that generates a .js file but then most of the script will be the same and I'm not sure how you can do code reuse in PowerShell. Am I on the right track?

[MaxShehovtsov]

Can you describe again why do we need this change? (I tried to follow microsoft/ApplicationInsights-aspnetcore#120 (comment) but it didn't really made sense to me)

[RehanSaeed]

I want to add application insights as a separate JavaScript file, rather than inline script. This is to enable Content Security Policy (CSP) HTTP header compatibility (Inline scripts are not allowed without adding a hash or nonce attribute with specially computed values) for better security. According to the link posted by @SergeyKanzhelev, this also helped the Glimpse team achieve better performance.

To achieve this I want to use the gulp-replace plugin to take the instrumentation key from the configuration file and insert it into the application insights JavaScript file in my ASP.NET 5 project template. I could grab a copy of the file but I would have to update it myself when Microsoft changes it. It would be nice if the file came straight from a NuGet package which can be updated.

It would be even better if Microsoft released a Bower package for application insights explicitly for this purpose. @SergeyKanzhelev @MaxShehovtsov, what is Microsoft's appetite for this? Especially as Gulp and Bower are where ASP.NET 5 is headed.

[MaxShehovtsov]

So once again, why prefix.html from nuget doesn't work?

[RehanSaeed]

I need a separate .js file. I could use the .html file but then I would have to strip out the <script> tags using yet another Gulp plugin.

[MaxShehovtsov]

Strip out what? :)

[SergeyKanzhelev]

<script> tag

[RehanSaeed]

I have updated the ASP.NET MVC Boilerplate project template where you can see my use case in action, please see this issue for more information.

[RehanSaeed]

I have spotted an applicationinsights-js bower package with the snippet.js file in it. I can now use that file in ASP.NET Core. This can be closed, unless someone finds this file useful in ASP.NET 4.6.

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.