Review lifecycle of services registered in the DI container

[joegoldman2]

Fixes #439.

[codecov-commenter]

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (5284757) 74.11% compared to head (243b7de) 74.16%.

Files	Patch %	Lines
Src/Fido2.AspNet/Fido2NetLibBuilderExtensions.cs	0.00%	5 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #459      +/-   ##
==========================================
+ Coverage   74.11%   74.16%   +0.05%     
==========================================
  Files         100      100              
  Lines        2712     2710       -2     
  Branches      443      443              
==========================================
  Hits         2010     2010              
+ Misses        598      596       -2     
  Partials      104      104              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[abergs]

I would need to think about what side effect this change could have before this is merged.

I welcome others analysis on this...

[abergs]

@iamcarbon Happy to hear if you have any thoughts

[iamcarbon]

Looks good. 👍

[abergs]

Thank you @joegoldman2 👍 Happy friday.