getsentry / raven-go

Sentry client in Go
https://sentry.io
BSD 3-Clause "New" or "Revised" License
561 stars 148 forks source link

Adding support for an array of fingerprints #71

Closed aheuermann closed 8 years ago

mattrobenolt commented 8 years ago

Oh, oops. I somehow missed this in the previous PR. :(

Thanks for fixing!

Sinjo commented 8 years ago

Sorry about the bug! The string worked against Sentry 7 (which we were running at the time), and I didn't check against Sentry 8. :(

mattrobenolt commented 8 years ago

@Sinjo That's because sentry 7 didn't support fingerprints. :)

It's all good.

Sinjo commented 8 years ago

It wasn't mega documented at the time, but we were using it from https://github.com/getsentry/sentry/blob/master/CHANGES#L255

mattrobenolt commented 8 years ago

Ah, I guess you're right. I thought we added that in 8. :( Hard to keep track at this point.

Sinjo commented 8 years ago

Hard to keep track at this point.

It's all good. Much :heart: for all the new features!