-
Notifications
You must be signed in to change notification settings - Fork 57
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Asp35 instrumentation to ignore Options requests and not use empty names #612
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. I had added a few questions, but nothing that blocks merging
...t/NewRelic/Agent/Extensions/Providers/Wrapper/Asp35/IntegratedPipeline/ExecuteStepWrapper.cs
Show resolved
Hide resolved
src/Agent/NewRelic/Agent/Extensions/Providers/Wrapper/Asp35/Shared/RouteNamingWrapper.cs
Show resolved
Hide resolved
tests/Agent/IntegrationTests/IntegrationTestHelpers/Assertions.cs
Outdated
Show resolved
Hide resolved
tests/Agent/IntegrationTests/IntegrationTests/RemoteServiceFixtures/WebApiAsyncFixture.cs
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changelog needs to be updated
Description
Resolves #463 and #186 with some updates to the
Asp35
instrumentation, adding some checks to ignoreOPTIONS
requests and to not replace transaction names with nothing.Testing
Included are two integration tests for validating that
Mvc3
andWebApi2
instrumentation is handling and naming these requests correctly.