[nodejs/typescript] App Insights samples missing bot properties

[daveta]

Sample information

1. Sample type: \samples\ and \solutions\
2. Sample language: nodejs and typescript
3. Sample name:
   21.luis-with-appinsights
   20.qna-with-appinsights
   Enterprise Template

Describe the bug

Not clear if something changed in restify, but any sample that uses Application Insights requires a small fix below:

// Create HTTP server
let server = restify.createServer();

// Enable the Application Insights middleware, which helps correlate all activity
// based on the incoming request.
server.use(restify.plugins.bodyParser());  // <== NEED THIS LINE
server.use(ApplicationInsightsWebserverMiddleware);

This enables the telemetry generated by Application Insights to use userId and sessionId that are bot specific (instead of web identifiers which don't make sense). It also adds some additional commonly used properties (ie, activityId, channelId, etc).

[bug]

[benbrown]

You should chat with @daveta because he also discovered this bug and may have fixed it elsewhere.

[CoHealer]

Removing Support resources post reassignment

[cleemullins]

@Kaiqb "We believe this is fixed as the samples have been re-done".

[daveta]

Samples now pass middleware to restify..