cloudflare / raven-lua

A Lua interface to Sentry
BSD 3-Clause "New" or "Revised" License
118 stars 52 forks source link

Add Support for addition Sentry Fields (Users, Request Data, Environment, Release) #40

Open cycomachead opened 3 years ago

cycomachead commented 3 years ago

Sorry this is one big PR. I've been meaning to try to upstream some changes for a while. If there's interest, consider this the start of a conversation — I can try to break this up as necessary.

Essentially, I've been using raven-lua pretty successfully in a Lapis (OpenResty) project. I've needed to extend this to support a few additional fields more common in traditional user-facing web apps.

DDevine commented 3 years ago

It isn't that big of a merge... And it looks good to me.

nevmerzhitsky commented 2 years ago

Why it wasn't merged, guys? :'(

nevmerzhitsky commented 2 years ago

@jdesgats consider merge of this, please

amo13 commented 1 year ago

Please merge