Skip to content

fix: enforce pause checks#21429

Open
mk0walsk wants to merge 3 commits intotaikoxyz:mainfrom
mk0walsk:pausegate
Open

fix: enforce pause checks#21429
mk0walsk wants to merge 3 commits intotaikoxyz:mainfrom
mk0walsk:pausegate

Conversation

@mk0walsk
Copy link

@mk0walsk mk0walsk commented Mar 4, 2026

This PR makes bridge pause checks actually block execution.

Several bridge flows were calling isBridgePaused().then(...), which does not gate the current async flow and can throw outside local try/catch. That means paused-state checks could be bypassed in practice.
I replaced those call sites with awaited guards (if (await isBridgePaused()) ...) in the bridge/approval paths and used early returns in UI handlers where throwing would become an unhandled async error.

I also added explicit BridgePausedError handling in handleBridgeError, so paused-state failures show a clear paused message instead of falling back to a generic unknown error.

@mk0walsk mk0walsk changed the title Enforce Pause Checks fix: enforce pause checks Mar 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants