-
-
Notifications
You must be signed in to change notification settings - Fork 248
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
Sentry SDK throws with dart compile js - cannot parse DSN (likely Flutter Web too) #1900
Comments
This is likely relevant: dart-lang/sdk#55058 |
interesting edge case, thx for the deep dive! I'd assume this is blocked by the dart issue? |
Correct. Someone one the Dart team said they would look at it. What do you guys usually do in these situations? Update docs eg. or wait for fix? |
In that case we would add it to our troubleshooting docs: https://docs.sentry.io/platforms/flutter/troubleshooting/ If you like you can also open up a PR for the site :) https://github.com/getsentry/sentry-docs/blob/master/docs/platforms/flutter/troubleshooting.mdx it should be fine to document it there, we unfortunately don't have a dart troubleshooting page |
@buenaflor this issue was marked as duplicate, but the original issue is resolved: dart-lang/sdk#49599. marking this for discussion |
I'll close this since the issue has been resolved in the dart sdk and I can't repro it locally (anymore) |
Platform
Dart Web
Obfuscation
Disabled
Debug Info
Disabled
Doctor
dart info
Version
7.16.1
Steps to Reproduce
Follow recommended sentry setup, provide a DSN to it that looks like:
Make sure that
dart2js
is set toO0
(Oh, zero) optimization level (other options don't seem to affect the output).I've done this via build.yaml
Run the app with either
webdev
orbuild_runner
.Expected Result
App to compile and run without errors.
❗ NOTE: other optimization levels, -O1, -O2, -O3 and -O4 all seem to generate proper output and there is NO error
Actual Result
The following exception is thrown:
This is likely NOT a sentry issue but an SDK one. I do believe this also affects Flutter Web.
Minimum repro code
Run the above code with
dart run build_runner serve web -r
with-O0
in build.yaml.The
@
character is much disliked by -O0.Are you willing to submit a PR?
Yes
The text was updated successfully, but these errors were encountered: