search

phptek / silverstripe-sentry

Flexible Sentry compatible bug aggregation client for Silverstripe applications.
BSD 3-Clause "New" or "Revised" License
11 stars 22 forks source link

Capture messages without timestamps in the title #78

Closed HJGreen closed 2 years ago

HJGreen commented 2 years ago

Allows the Sentry UI to properly group related issues. The formatted property includes a timestamp, which causes Sentry to log them as separate issues.

See the attached screenshot for an example. There are two missing CSS files which Silverstripe is trying to link into the page. The first four entries show the current behaviour where each time the page is loaded, a new Issue is logged in Sentry due to the timestamp.

The bottom two rows are with the PR applied - similar events are grouped with the original.

Screen Shot 2021-09-28 at 5 07 09 PM

phptek commented 2 years ago

@HJGreen thanks for this - using "message" actually sorts the problem. In the process of looking at this, I realised that exceptions were also logging twice for each message. I have fixed this too - currently in master.

phptek commented 2 years ago

@HJGreen FYI, you can use tag 4.0.5 which incorporates this fix.