Switch default dandi instance to dandi-api based on redirector

[jwodder]

Closes #541.

[codecov]

Codecov Report

Merging #565 (53b2021) into master (6a5f285) will increase coverage by 0.02%.
The diff coverage is 88.57%.

@@            Coverage Diff             @@
##           master     #565      +/-   ##
==========================================
+ Coverage   83.70%   83.73%   +0.02%     
==========================================
  Files          62       62              
  Lines        6530     6559      +29     
==========================================
+ Hits         5466     5492      +26     
- Misses       1064     1067       +3     

Flag	Coverage Δ
unittests	83.73% <88.57%> (+0.02%)	⬆️

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

Impacted Files	Coverage Δ
dandi/dandiarchive.py	85.26% <78.57%> (-0.05%)	⬇️
dandi/utils.py	80.19% <80.00%> (-0.14%)	⬇️
dandi/delete.py	96.99% <100.00%> (ø)
dandi/tests/test_dandiarchive.py	100.00% <100.00%> (ø)
dandi/tests/test_utils.py	98.65% <100.00%> (+0.03%)	⬆️

---

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 6a5f285...53b2021. Read the comment docs.

[yarikoptic]

could we please retain robustness of .get to provide None? or what would be the benefit of client crashing here if redirector again just drops a field instead of explicitly annotating with null?

[jwodder]

Done.

[yarikoptic]

Thank you! next one to touch this piece of code is to reduce duplication. Let's proceed

[yarikoptic]

besides one comment (I would prefer .get over [] unless there is a good reason), looks good to me. Should be taken out of the Draft and to be merged