Merge remote-tracking branch 'upstream/develop' into relay

* upstream/develop:
  Remove flow assert: (5009)
  Update list of maintainers: (4984)

Author: ximinez

CONTRIBUTING.md

@@ -200,7 +200,6 @@ existing maintainer without a vote.
 * [JoelKatz](https://github.com/JoelKatz) (Ripple)
 * [manojsdoshi](https://github.com/manojsdoshi) (Ripple)
 * [n3tc4t](https://github.com/n3tc4t) (XRPL Labs)
-* [Nik Bougalis](https://github.com/nbougalis)
 * [nixer89](https://github.com/nixer89) (XRP Ledger Foundation)
 * [RichardAH](https://github.com/RichardAH) (XRPL Labs + XRP Ledger Foundation)
 * [seelabs](https://github.com/seelabs) (Ripple)

src/ripple/app/paths/impl/StrandFlow.h

@@ -834,7 +834,13 @@ flow(
     {
         if (actualOut > outReq)
         {
-            assert(0);
+            // Rounding in the payment engine is causing this assert to
+            // sometimes fire with "dust" amounts. This is causing issues when
+            // running debug builds of rippled. While this issue still needs to
+            // be resolved, the assert is causing more harm than good at this
+            // point.
+            // assert(0);
+
             return {tefEXCEPTION, std::move(ofrsToRmOnFail)};
         }
         if (!partialPayment)