MS_SubRoutes are not affecting Transaction names.

APM Agent version

The version of the Elastic.Apm nuget package used: master branch directly, AspNetFullFrameworkSampleApp

Environment

Operating system and version: Windows Version 10.0.19042.804

.NET Framework/Core name and version (e.g. .NET 4.6.1, NET Core 3.1.100) : .NET 4.6.1

Application Target Framework(s) (e.g. net461, netcoreapp3.1): net461

Describe the bug

Transaction names are generated from request.Unvalidated.Path for WebApi routes that have [Route] attribute on it.

To Reproduce

Steps to reproduce the behavior:

1. Use AspNetFullFrameworkSampleApp from master branch (set default apm and kibana from for example Quick start guide
2. Find AspNetFullFrameworkSampleApp.Controllers.WebApiController
3. Add this sample controller method:

[RoutePrefix("api/webapi")]
public class WebApiController : ApiController
{
	...
	[HttpGet]
	[Route("{someId}")]
	public IHttpActionResult AskMe(Guid someId) => Ok($"This is an example response from a web api controller: {someId}");
	...

1. Launch AspNetFullFrameworkSampleApp
2. Use for example postman to request endpoint with dummy guid:
3. Find call in Kibana APM

Expected behavior

Transaction name is taken from subroute route template. Example fix would be in Elastic.Apm.Model.Transaction.GetNameFromRouteContext(IDictionary<string, object> routeValues) you can add another if statement (around line 546):

else if (routeValues.TryGetValue("MS_SubRoutes", out var template))
{
	var subroute = ((IEnumerable<IHttpRouteData>)template).FirstOrDefault();
	if(subroute != null)
	{
		name = subroute.Route.RouteTemplate;
	}
}

Actual behavior

Guid is used as transaction name:

[russcam]

Thanks for opening @kszymanski. This seems like a reasonable change to include

[insanity13]

Unfortunately, I have not found a way to get information about the route in my case (in Request End).

• .NET 4.7.2
• ASP.NET Web App using OWIN Startup
• • System.Web.Http.Owin, Version=5.2.7
• • Microsoft.AspNet.WebApi, Version=5.2.7

Found only one option: to register the ActionFilter and get a template from ControllerContext. In this place, the template is correct.