search

MindscapeHQ / raygun4net

Raygun provider for .NET
https://raygun.com
MIT License
125 stars 91 forks source link

Fix missing cookie collection in ASP.NET Core provider #458

Closed j5alive closed 2 years ago

j5alive commented 2 years ago

The check for request.HasFormContentType when trying to get cookies from the request should not be there as the Form content has no bearing on a request containing cookies or not. It looks like this was copied over from the original implementation.

I've also added a message to the cookies list if this method fails in line with the other methods in this class.

j5alive commented 2 years ago

@QuantumNightmare any chance you could review this please?

QuantumNightmare commented 2 years ago

This fix has been released in version 6.6.5 of the NuGet package now.