Skip to content

Conversation

@squadgazzz
Copy link
Contributor

Description

After the CoWSwapEthFlow SC was migrated to alloy, we've started receiving the following errors:

2025-10-14T16:46:08.171Z  WARN refunder: Error while refunding ethflow orders: local usage error: Missing signing credential for 0x0214aE5fD178986fA18ff792e0b995Dc6a78cD56  
Caused by:     Missing signing credential for 0x0214aE5fD178986fA18ff792e0b995Dc6a78cD56

That basically means that our refunder doesn't work.

The reason for that is that the refunder's private key wasn't set properly in the alloy provider. This PR fixes it.

How to test

Probably not easy.

Copilot AI review requested due to automatic review settings October 15, 2025 13:01
@squadgazzz squadgazzz requested a review from a team as a code owner October 15, 2025 13:01
@squadgazzz squadgazzz added the hotfix Labels PRs that should be applied into production right away label Oct 15, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes the refunder failing to sign transactions after the CoWSwapEthFlow migration to Alloy by attaching the correct signer to the Alloy provider used for contract calls.

  • Import ProviderSignerExt and TryIntoAlloyAsync to derive an Alloy signer from the configured account.
  • Construct the CoWSwapEthFlow instance with a provider bound to the signer via with_signer(...).

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@squadgazzz squadgazzz enabled auto-merge (squash) October 15, 2025 13:12
@squadgazzz squadgazzz merged commit 55f1ce5 into main Oct 15, 2025
17 checks passed
@squadgazzz squadgazzz deleted the fix-refunder branch October 15, 2025 13:18
@github-actions github-actions bot locked and limited conversation to collaborators Oct 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

hotfix Labels PRs that should be applied into production right away

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants