mlsmaycon d3e0ee8547 [proxy] Fail a mapping update whose chains would not install ...

Publishing the chain before the route only helps if the chain is there.
rebuildMiddlewareChains logged its error and returned, so a failed rebuild
still fell through to AddMapping and published a route over chains that were
never installed — served with no policy enforcement and no metering, which
is the outcome the ordering change exists to prevent.

Report the error instead. The caller already unwinds a failed setup, so the
service stays unpublished rather than reachable and uncounted. An unset
middleware manager is still not an error: that is a deployment without
middleware, not a failure to install it.

2026-08-23 07:26:27 +00:00

78 KiB

Raw Permalink Blame History

View Raw