Introduce scope stack abstraction

[Tyrrrz]

Closes #1106
Closes #628

Related to #221, #448, #505

Not entirely sure how to test this. Ideas @bruno-garcia?

[codecov-commenter]

Codecov Report

Merging #1124 (248f929) into main (d7a96a3) will increase coverage by 0.01%.
The diff coverage is 88.46%.

@@            Coverage Diff             @@
##             main    #1124      +/-   ##
==========================================
+ Coverage   80.64%   80.65%   +0.01%     
==========================================
  Files         197      200       +3     
  Lines        6592     6612      +20     
  Branches     1455     1460       +5     
==========================================
+ Hits         5316     5333      +17     
- Misses        802      804       +2     
- Partials      474      475       +1     

Impacted Files	Coverage Δ
....AspNet/Internal/HttpContextScopeStackContainer.cs	0.00% <0.00%> (ø)
src/Sentry/Internal/SentryScopeManager.cs	94.66% <90.90%> (-0.99%)	⬇️
src/Sentry.AspNet/SentryAspNetOptionsExtensions.cs	90.90% <100.00%> (+0.90%)	⬆️
src/Sentry/Internal/Hub.cs	62.37% <100.00%> (+0.76%)	⬆️
...ternal/ScopeStack/AsyncLocalScopeStackContainer.cs	100.00% <100.00%> (ø)
...y/Internal/ScopeStack/GlobalScopeStackContainer.cs	100.00% <100.00%> (ø)
src/Sentry/SentryOptions.cs	89.44% <100.00%> (+0.20%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d7a96a3...248f929. Read the comment docs.

[bruno-garcia]

I like where this is going!

[bruno-garcia]

Not entirely sure how to test this. Ideas @bruno-garcia?

I think the direction you're going for a test for each approach is a good one. Let me take another look once it's out of draft to propose some integration/unit tests

[Tyrrrz]

@bruno-garcia what do you think about the general implementation by the way? I know you said there are a lot of potential edge-cases/hidden issues, so I'm somewhat skeptical about my solution given how easy it was to implement 😅

It could benefit to put some more responsibility inside IScopeStackContainer, but I decided to go with the easiest approach for now.

[bruno-garcia]

Thanks!

[bruno-garcia]

We need to ship this and add some docs