Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Hubs/Scopes Merge 24 - Use new API in Spring integrations #3348

Merged
merged 30 commits into from
Apr 22, 2024
Merged

Hubs/Scopes Merge 24 - Use new API in Spring integrations #3348

merged 30 commits into from
Apr 22, 2024

Conversation

adinauer
Copy link
Member

@adinauer adinauer commented Apr 12, 2024 •
edited
Loading

#skip-changelog

📜 Description

💡 Motivation and Context

💚 How did you test it?

📝 Checklist

  • I reviewed the submitted code.
  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.

🔮 Next steps

adinauer and others added 29 commits March 27, 2024 16:30
@adinauer
replace hub with scopes
305baf5
@adinauer
Add Scopes
95f5e1b
@adinauer
Introduce IScopes interface.
27f2398
@adinauer
Replace IHub with IScopes in core
ce3c14f
@adinauer
Replace IHub with IScopes in android core
ce615f4
@adinauer
Replace IHub with IScopes in android integrations
22ddc00
@adinauer
Replace IHub with IScopes in apollo integrations
305c217
@adinauer
Replace IHub with IScopes in okhttp integration
da927bc
@adinauer
Replace IHub with IScopes in graphql integration
8279276
@adinauer
Replace IHub with IScopes in logging integrations
9bfc086
@adinauer
Replace IHub with IScopes in more integrations
b998e50
@adinauer
Replace IHub with IScopes in OTel integration
739827a
@adinauer
Replace IHub with IScopes in Spring 5 / Spring Boot 2 integrations
69f2d63
@adinauer
Replace IHub with IScopes in Spring 6 / Spring Boot 3 integrations
792d482
@adinauer
Replace IHub with IScopes in samples
9bcbce6
@adinauer
Merge branch 'feat/hsm-13-replacements-in-samples' into feat/hubs-sco…
3f25a4b 
…pes-merge-2-add-scopes
@adinauer
gitscopes -> github
d6fb40a
@adinauer
Replace ThreadLocal with ScopesStorage
7752bcc
@adinauer
Move client and throwable to span map to scope
1e329c5
@adinauer
Add global scope
b0d89ae
@adinauer
use global scope in Scopes
cdd414a
@adinauer
Implement pushScope popScope and withScope for Scopes
98da9ff
@adinauer
Add pushIsolationScope; add fork methods to ISCope
2d26033
@adinauer
Use separate scopes for current, isolation and global scope; rename m…
bbb6700 
…ainScopes to rootScopes
@adinauer
Allow controlling which scope configureScope uses
c714b21
@adinauer
Combine scopes
a474402
@adinauer
Use new API for CRONS integrations
ae93e33
@adinauer
Add lifecycle helper
b01298b
@adinauer
Change spring integrations to use new API
b64e688
@adinauer adinauer requested a review from romtsn as a code owner April 12, 2024 12:06
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Apr 12, 2024 •
edited
Loading

Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 855ebfe

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Apr 12, 2024 •
edited
Loading

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 355.84 ms 441.84 ms 85.99 ms
Size 1.70 MiB 2.28 MiB 595.91 KiB

Previous results on branch: feat/hsm-24-spring-integrations

Startup times

Revision Plain With Sentry Diff
ddf2c22 377.02 ms 434.83 ms 57.81 ms

App size

Revision Plain With Sentry Diff
ddf2c22 1.70 MiB 2.28 MiB 595.91 KiB

Base automatically changed from feat/hsm-23-cron-integrations to 8.x.x April 22, 2024 13:13
@adinauer adinauer merged commit 9a64a0b into 8.x.x Apr 22, 2024
5 of 19 checks passed
@adinauer adinauer deleted the feat/hsm-24-spring-integrations branch April 22, 2024 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lbloder lbloder lbloder left review comments

@markushi markushi markushi left review comments

@romtsn romtsn Awaiting requested review from romtsn romtsn is a code owner

@stefanosiano stefanosiano Awaiting requested review from stefanosiano stefanosiano is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@adinauer @markushi @lbloder