Uh oh!
There was an error while loading. Please reload this page.
ref(core): Improve event mechanism for supabase integration - #17286
Conversation
| return res; | ||
| }, | ||
| (err: Error) => { | ||
| // TODO: shouldn't we capture this error? |
There was a problem hiding this comment.
@onurtemizkan not sure if this is me missing something or if we should call captureException here. Can you take a look?
There was a problem hiding this comment.
Hmm, looks like this comes from the original implementation we ported:
I don't see a reason why we should not capture this, though.
There was a problem hiding this comment.
thanks! Probably just an oversight then. I'll make a separate PR adding a captureException call here.
| captureException(res.error, { | ||
| mechanism: { | ||
| handled: false, | ||
| type: 'auto.db.supabase.auth', |
There was a problem hiding this comment.
thesetype entries are one level more specific than the trace origin on the spans (auto.db.supabase). I think this is fine.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
ref: #17250
ref: #17212