Skip to content

Conversation

@maks
Copy link

@maks maks commented Jan 10, 2020

This gets the contexts branch updated to current Sentry plugin master branch, adds a test for the contexts functionality and fixes a couple of small bugs identified by the test.

filiph and others added 13 commits August 29, 2019 21:25
…try#42)

1. dsn may specify non-regular port
2. dsn may contains some paths before the projectID
* Add pedantic and analysis options
* Fix analysis warnings
- [x] Refactor to support browser
  + `SentryClient` from `package:sentry/sentry.dart` with conditional import
  + `SentryBrowserClient` for web from `package:sentry/browser_client.dart`
  + `SentryIOClient` for VM and Flutter from `package:sentry/io_client.dart`
- [x] Write test for browser
- [x] Working Angular Sentry packages => https://github.com/leftyio/angular_sentry

Main implementation differences are:
- gzip compression is provided implicitly by the browser
- javascript stacktraces need to be prefix with `window.location.origin` to be resolve by Sentry
@wilkomanger wilkomanger merged commit f631cc2 into wilkomanger:contexts Jan 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants