-
-
Notifications
You must be signed in to change notification settings - Fork 277
Refacto : add a Transport class #123
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
40 commits
Select commit
Hold shift + click to select a range
a10d740
remove the SentryResponse class
rxlabz 623c904
refactor : add a Transport class
rxlabz 0dcb613
refactor : move the clock initialization to Transport
rxlabz c97c98b
- refactor : applyScope
rxlabz 477fdac
changelog
rxlabz b8fb322
fix merge conflicts
rxlabz fd92e3f
Merge branch 'feature/unified-api' into ref/transport
rxlabz 0944c68
move the headers definition strategy to transport
rxlabz 46829fc
- Instantiate Transport from options
rxlabz 01d4e19
handle event dropping in event processors
rxlabz fbd001e
fix client._applyScope : merge event and scope tags and extra
rxlabz 4d39531
remove client.toString()
rxlabz 34fb528
fix a test
rxlabz 2437fa4
Merge branch 'feature/unified-api' into ref/transport
rxlabz 57798f5
handle the event serialization in the transport layer
rxlabz 789e154
Merge remote-tracking branch 'origin/feature/unified-api' into ref/tr…
rxlabz 1aa05d6
remove stackFrameFilter references
rxlabz 9895142
Merge remote-tracking branch 'origin/feature/unified-api' into ref/tr…
rxlabz cb5e309
add a CredentialBuilder
rxlabz e563d2c
- headers configuration
rxlabz 71b53c8
- fix scope.level precedence
rxlabz be3f0a2
buildHeaders and fix timestamp test
rxlabz 43b798c
- add a isWeb helper to define the event.platform
rxlabz 19dc91d
feat: apply sample rate
marandaneto cec1f21
changelog
marandaneto 909d4f9
- add a isWeb helper to define the event.platform and sdkName
rxlabz 8b600dc
- add a isWeb helper to define the event.platform and sdkName
rxlabz 4ff58a6
Merge remote-tracking branch 'origin/feat/sample-rate' into ref/trans…
rxlabz 5493e19
fix
marandaneto 621d6c4
update tests
rxlabz e11f653
Merge remote-tracking branch 'origin/feat/sample-rate' into ref/trans…
rxlabz 88526e2
remove the platform arg from Client ctor
rxlabz 796b633
set client.options private
rxlabz fb8b9e0
remove event.platform redefinition
rxlabz 1dbcba4
set options.sdk if null
rxlabz 8b56bce
remove fields from Transport Api
rxlabz daeb29e
private version consts
rxlabz 098d58c
remove transport from the Client ctor
rxlabz 2833a94
remove options.environmentAttributes
rxlabz 35671d4
event processing : platform
rxlabz File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
fix merge conflicts
- Loading branch information
commit b8fb3227bab48016784a6ac6519ae299da90b62d
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
still missing the execution of the beforeSendCallback ->
https://github.com/getsentry/sentry-java/blob/5ebd49cbfd17eb66175e9247db6d27e4df84a56b/sentry/src/main/java/io/sentry/SentryClient.java#L97-L103