Split AppState into logical components

[gswirski]

Motivation

Simplify main.rs file and provide finer grained locks.

Solution

Split state into smaller components that can be locked separately.

I erred on the side of explicit dependencies which means we have lots of Axum extensions now.

Alternatively, we could have a single Arc<AppState> shared across endpoints with many RwLockss and AtomicInts inside.

PR Checklist

• Added Tests
• Added Documentation
• Updated the changelog

[codecov-commenter]

Codecov Report

Merging #22 (3327cbb) into master (00b4bdb) will decrease coverage by 0.19%.
The diff coverage is 46.12%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master      #22      +/-   ##
==========================================
- Coverage   23.85%   23.65%   -0.20%     
==========================================
  Files          17       20       +3     
  Lines        5157     5285     +128     
==========================================
+ Hits         1230     1250      +20     
- Misses       3927     4035     +108     

Flag	Coverage Δ
test	23.65% <46.12%> (-0.20%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
src/api/v1/auth.rs	0.00% <0.00%> (ø)
src/api/v1/info.rs	0.00% <0.00%> (ø)
src/main.rs	2.40% <0.00%> (-29.05%)	⬇️
src/oauth.rs	0.00% <0.00%> (ø)
src/util.rs	14.28% <0.00%> (ø)
src/lobby.rs	83.13% <83.13%> (ø)
src/api/v1/lobby.rs	78.33% <91.66%> (-0.77%)	⬇️
src/api/v1/contribute.rs	91.97% <95.83%> (-0.58%)	⬇️
... and 2 more

---

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 00b4bdb...3327cbb. Read the comment docs.

[kevaundray]

In terms of best practice, not just for Axum, having a lock within a lock is a code smell. So I think splitting it up would be best