Skip to content
This repository has been archived by the owner on Dec 19, 2018. It is now read-only.

Move MVC's TelemetrySource registration to Hosting #353

Closed
Eilon opened this issue Sep 14, 2015 · 1 comment
Closed

Move MVC's TelemetrySource registration to Hosting #353

Eilon opened this issue Sep 14, 2015 · 1 comment
Assignees
@Tratcher
Labels
3 - Done bug
Milestone
1.0.0-beta8

Comments

@Eilon
Copy link
Member

Eilon commented Sep 14, 2015

This avoids multiple components each trying to register the services themselves.

Source codes here: https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNet.Mvc.Core/DependencyInjection/MvcCoreServiceCollectionExtensions.cs#L139-L148
@Eilon Eilon added this to the 1.0.0-beta6 milestone Sep 14, 2015
@Tratcher Tratcher modified the milestones: 1.0.0-beta8, 1.0.0-beta6 Sep 14, 2015
@Eilon
Copy link
Member Author

Eilon commented Sep 14, 2015

No, get it done in beta 6 :trollface:

Tratcher added a commit that referenced this issue Sep 16, 2015
@Tratcher
#353,#354 Add telemetry for begin/end request and unhandled exceptions.
6758010
@Tratcher Tratcher mentioned this issue Sep 16, 2015
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@Tratcher Tratcher

Labels
3 - Done bug
Projects
None yet
Milestone
1.0.0-beta8
Development

No branches or pull requests
2 participants
@Eilon @Tratcher