-
-
Notifications
You must be signed in to change notification settings - Fork 234
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: windows native error & obfuscation support #2286
base: main
Are you sure you want to change the base?
Conversation
|
9766931
to
fd2d073
Compare
This reverts commit 1c7b63d.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
overall looks good 👍
since this is a big PR I'll do another pass later
@@ -91,6 +91,7 @@ class SentryStackFrame { | |||
/// The "package" the frame was contained in. | |||
final String? package; | |||
|
|||
// TODO what is this? doesn't seem to be part of the spec https://develop.sentry.dev/sdk/event-payloads/stacktrace/ | |||
final bool? native; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hm not sure if this is even used in the sentry product
extension on String { | ||
binding.sentry_value_u toNativeValue() { | ||
final cValue = toNativeUtf8(); | ||
final result = SentryNative.native.value_new_string(cValue.cast()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is there a chance when cValue.cast()
throws an exception that cValue
is never freed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't believe cast() can throw
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
but I'll check
📜 Description
This adds support for:
💡 Motivation and Context
split-debug-info
for Flutter Desktop (Linux and Windows) #433💚 How did you test it?
Added tests
📝 Checklist
sendDefaultPii
is enabled🔮 Next steps