chore: ignore uncommitted dbtx writes for endpoint audits

[bradleystachurski]

Fixes #3750

Please refer to this comment for reasoning behind this approach. #3750 (comment)

Reproduced the WARN log reported in the associated issue and verified these changes no longer log a warning when running fedimint-cli --our-id 0 --password pass admin audit in mprocs.

Before

nix@lenovo-m75q:/tmp/nix-shell.yafZfU/devimint-175/logs$ rg 'DatabaseTransaction has writes and has not called commit' fedimintd-0.log
78:2023-11-28T05:23:39.844034Z  WARN db: DatabaseTransaction has writes and has not called commit. location=   0: <fedimint_core::db::CommitTracker as core::ops::drop::Drop>::drop

After

nix@lenovo-m75q:/tmp/nix-shell.yafZfU/devimint-68797/logs$ rg 'DatabaseTransaction has writes and has not called commit' fedimintd-0.log

[bradleystachurski]

Flakey code coverage in CI. I'll keep kicking the tires tomorrow (already captured in flakey CI tracking issue).

[codecov]

Codecov Report

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

Comparison is base (681e6cc) 57.67% compared to head (7420ff3) 57.68%.
Report is 4 commits behind head on master.

Files	Patch %	Lines
fedimint-server/src/net/api.rs	0.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3756      +/-   ##
==========================================
+ Coverage   57.67%   57.68%   +0.01%     
==========================================
  Files         193      193              
  Lines       41976    41990      +14     
==========================================
+ Hits        24208    24222      +14     
  Misses      17768    17768              

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

[fedimint-backports]

Successfully created backport PR for releases/v0.2:

• [Backport releases/v0.2] chore: ignore uncommitted dbtx writes for endpoint audits #3765

[justinmoon]

Thanks @bradleystachurski