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

Invalid data (invalid character '{') in path #1177

Closed
Cyral opened this issue Aug 27, 2021 · 3 comments · Fixed by #1183
Closed

Invalid data (invalid character '{') in path #1177

Cyral opened this issue Aug 27, 2021 · 3 comments · Fixed by #1183
Labels
ASP.NET Core Bug Something isn't working

Comments

@Cyral
Copy link

Cyral commented Aug 27, 2021

Environment

How do you use Sentry?
Sentry SaaS

Which SDK and version?
.NET 5.0.8, Sentry.AspNetCore 3.8.3

Steps to Reproduce

Installed Sentry.AspNetCore according to guide:

In Program.cs CreateDefaultBuilder:

.ConfigureWebHostDefaults(webBuilder => {
    webBuilder.UseSentry(o => {
        o.Dsn = "https://xxx.ingest.sentry.io/xxx";
        // o.Debug = true;
        o.SendDefaultPii = true;
        o.TracesSampleRate = 0.1;
        o.AddExceptionFilterForType<ApiException>();
});

In Startup.cs Configure method:

app.UseRouting();
app.UseSentryTracing();

Expected Result

Correct path should show in issue

Actual Result

image

@bruno-garcia
Copy link
Member

To be quite honest I saw this issue while dog fooding the SDK and I'm not sure yet what integration is adding this tag. Would appreciate some help pinning down this bug.

@bruno-garcia bruno-garcia added ASP.NET Core Bug Something isn't working labels Aug 27, 2021
@lucas-zimerman
Copy link
Collaborator

It comes from
Scope.cs Apply TState
image
one of the object keys is called {OriginalFormat}

public void Apply(object state) => Options.SentryScopeStateProcessor.Apply(this, state);

Perhaps adding a filter to remove the brackets from the keys inside of SentryScopeStateProcessor os simply filtering that {OriginalFormat} key

@bruno-garcia
Copy link
Member

We can filter out this one for now and ship a hotfix

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