Skip to content
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

Api name not being catpured. #1155

Closed
michaelakin opened this issue Aug 9, 2021 · 5 comments · Fixed by #1198
Closed

Api name not being catpured. #1155

michaelakin opened this issue Aug 9, 2021 · 5 comments · Fixed by #1198
Assignees
Labels
ASP.NET Core Bug Something isn't working

Comments

@michaelakin
Copy link

Environment

How do you use Sentry?
Sentry SaaS (sentry.io)

Which SDK and version?
.net core 3.8.3

Steps to Reproduce

  1. Did a post command to our wepapi.

Expected Result

should have captured the api name as well as the path

https://ourservername.com/UserAPI/v1/Authentication/AuthenticateUser

Actual Result

it only has references to the server and path, but not the api

it only shows this: v1/Authentication/AuthenticateUser

What actually happened. Maybe a screenshot/recording? Maybe some logs?

image

@lucas-zimerman
Copy link
Collaborator

Hi @michaelakin , could you please share how are you defining the route? And, do you recall if the same behavior happened on a previous version of the SDK?

@michaelakin
Copy link
Author

Sure, we have this on the controller

    [Route("v1/[controller]")]
    [ApiController]

And this on the method

        [HttpPost]
        [Route("AuthenticateUser")]
        public async Task<IActionResult> AuthenticateUser([FromBody] AuthUser user)

This is our first version to use of the sdk, so not sure if it ever worked.

We have multiple projects on the same web server and this does not seem to capture the name of the project.

Thanks!

@lucas-zimerman
Copy link
Collaborator

Oh, just to be sure.
You expected the UI to show the below path?

POST /UserAPI/v1/Authentication/AuthenticateUser

Where UserAPI is your path base?

@lucas-zimerman lucas-zimerman added the Bug Something isn't working label Aug 24, 2021
@michaelakin
Copy link
Author

That would be nice and easier to go to the right project in our system.

@bruno-garcia
Copy link
Member

@michaelakin is this something you'd be keen to contribute a PR to fix? We'll try to get to it but there's a bit of a backlog

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ASP.NET Core Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants