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.

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

Explicitly set the default for span creation to non-private #998

Merged
merged 1 commit into from
Jun 26, 2024

Conversation

bidetofevil
Copy link
Collaborator

@bidetofevil bidetofevil commented Jun 26, 2024

Goal

Make interfaces default to creating spans that are non-private. This is already the case for the create and start span APIs, but the other methods still defaulted to pairing private with internal, which can be confusing.

In practice, the only change is that network request spans are now non-private, which they should've been anyway. All other changes are removing explicit parameters that match the default, which doesn't change existing behaviour

Testing

Modified existing tests to verify the default. Did not add additional tests for network spans because that's just using the default now.

Copy link

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Manifest Files

Copy link
Collaborator Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @bidetofevil and the rest of your teammates on Graphite Graphite

Copy link

codecov bot commented Jun 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.00%. Comparing base (42639be) to head (1e3eeac).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #998   +/-   ##
=======================================
  Coverage   78.99%   79.00%           
=======================================
  Files         416      416           
  Lines       11052    11045    -7     
  Branches     1764     1764           
=======================================
- Hits         8731     8726    -5     
+ Misses       1540     1538    -2     
  Partials      781      781           
Files Coverage Δ
...bracesdk/capture/startup/AppStartupTraceEmitter.kt 70.16% <ø> (-1.04%) ⬇️
...id/embracesdk/internal/spans/EmbraceSpanFactory.kt 100.00% <ø> (ø)
...android/embracesdk/internal/spans/EmbraceTracer.kt 96.96% <100.00%> (+0.41%) ⬆️
...e/android/embracesdk/internal/spans/SpanService.kt 96.87% <100.00%> (ø)
...esdk/internal/spans/UninitializedSdkSpanService.kt 75.00% <100.00%> (+1.98%) ⬆️
...dk/network/logging/EmbraceNetworkLoggingService.kt 96.42% <ø> (-0.13%) ⬇️

@bidetofevil bidetofevil marked this pull request as ready for review June 26, 2024 18:39
@bidetofevil bidetofevil requested a review from a team as a code owner June 26, 2024 18:39
@bidetofevil bidetofevil requested a review from priettt June 26, 2024 18:39
@bidetofevil bidetofevil merged commit 8e95028 into master Jun 26, 2024
4 checks passed
@bidetofevil bidetofevil deleted the hho/span-default-privacy branch June 26, 2024 21:30
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.

2 participants