Skip to content

Commit 561e89a

Browse files
authored
Merge pull request #50 from nuxt-community/bridge
fix: support nuxt bridge
2 parents 09af494 + cac29df commit 561e89a

File tree

4 files changed

+201
-168
lines changed

4 files changed

+201
-168
lines changed

lib/appinsights.server.js

+9-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,12 @@
1+
import * as applicationinsights from 'applicationinsights'
2+
13
export default function (ctx, inject) {
24
// Inject AppInsights to the context as $appInsights
3-
inject('appInsights', process.appInsights || {})
5+
const config = <%= serialize(options.config) %>
6+
const appInsightsServer = applicationinsights.setup(<%= serialize(options.instrumentationKey) %>)
7+
applicationinsights.defaultClient.config = config
8+
if (optionsServer.initialize) {
9+
appInsightsServer.start()
10+
}
11+
inject('appInsights', applicationinsights || {})
412
}

lib/module.js

+9-3
Original file line numberDiff line numberDiff line change
@@ -60,22 +60,28 @@ module.exports = function appInsights (moduleOptions) {
6060
const optionsServer = deepMerge.all([options, privateRuntimeConfig])
6161

6262
const appInsightsServer = AppInsights.setup(optionsServer.instrumentationKey)
63+
const config = { ...AppInsights.defaultClient.config, ...optionsServer.serverConfig }
6364

64-
AppInsights.defaultClient.config = { ...AppInsights.defaultClient.config, ...optionsServer.serverConfig }
65+
AppInsights.defaultClient.config = config
6566
// Initialize AppInsights
6667
if (optionsServer.initialize) {
6768
appInsightsServer.start()
6869
}
6970
const appInsightsClient = AppInsights.defaultClient
7071

71-
process.appInsights = appInsightsClient
7272
logger.success('Started logging errors to AppInsights')
7373

7474
this.addPlugin({
7575
src: path.resolve(__dirname, 'appinsights.server.js'),
7676
fileName: 'appinsights.server.js',
77-
mode: 'server'
77+
mode: 'server',
78+
options: {
79+
instrumentationKey: optionsServer.instrumentationKey,
80+
config,
81+
initialize: optionsServer.initialize
82+
}
7883
})
84+
7985
this.nuxt.hook('render:setupMiddleware', app => app.use(requestHandler(appInsightsClient)))
8086
this.nuxt.hook('render:errorMiddleware', app => app.use(errorHandler(appInsightsClient)))
8187
this.nuxt.hook('generate:routeFailed', ({ errors }) => {

package.json

+2-2
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,8 @@
2222
"nuxt": "node --inspect=0.0.0.0 node_modules/nuxt-edge/bin/nuxt test/fixture"
2323
},
2424
"dependencies": {
25-
"applicationinsights": "^2.1.2",
26-
"@microsoft/applicationinsights-web": "^2.6.3",
25+
"applicationinsights": "^2.3.3",
26+
"@microsoft/applicationinsights-web": "^2.8.5",
2727
"consola": "^2.15.0",
2828
"deepmerge": "^4.2.2"
2929
},

0 commit comments

Comments
 (0)