Techdebt: S3: Untangle GET-methods

[bblommers]

No logical changes - this just untangles the massive _key_response_get-method in S3Response.

The end-goal here is to get rid of the if ".." in querystring.. spaghetti altogether, and let our regular dispatch method handle the dispatching, but there are several steps that need to be handled first:

1. untangle POST/PUT/DELETE requests
2. ensure dispatch() can handle querystrings
3. ensure dispatch() knows and can handle the difference between virtual-host and path-style URL's
4. change s3/urls.py to use dispatch
5. and then we can remove all if ".." in querystring sections

[codecov]

Codecov Report

Attention: Patch coverage is 97.37991% with 6 lines in your changes missing coverage. Please review.

Project coverage is 94.38%. Comparing base (fc60bd1) to head (d280ac7).
Report is 4 commits behind head on master.

Files	Patch %	Lines
moto/s3/responses.py	97.29%	6 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #8000   +/-   ##
=======================================
  Coverage   94.38%   94.38%           
=======================================
  Files        1130     1130           
  Lines       96587    96633   +46     
=======================================
+ Hits        91159    91205   +46     
  Misses       5428     5428           

Flag	Coverage Δ
servertests	28.98% <47.16%> (+0.10%)	⬆️
unittests	94.35% <97.37%> (+<0.01%)	⬆️

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

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

[github-actions]

This is now part of moto >= 5.0.14.dev6