Skip to content

Feat/sealed bid auction/only accept prices from connected solvers#229

Merged
kss-t1 merged 2 commits intocanaryfrom
feat/sealed-bid-auction/only-accept-prices-from-connected-solvers
Jan 9, 2026
Merged

Feat/sealed bid auction/only accept prices from connected solvers#229
kss-t1 merged 2 commits intocanaryfrom
feat/sealed-bid-auction/only-accept-prices-from-connected-solvers

Conversation

@kss-t1
Copy link
Member

@kss-t1 kss-t1 commented Jan 9, 2026

Description

Only considers prices from authenticated solvers when running an auction

Motivation and Context

We need it because if solver disconnected (for whatever reason, could be a temporary connectivity issue) and we'd still consider his prices for auctioning, we have a risk of using his prices, running and auction and notifying solver on a topic.

In such case, since solver is not connected - he would never learn he won auction and would never solve it.

After this change, solver's prices will be considered as soon as he reconnects and is therefore able to receive notification about won auction.

How Has This Been Tested?

Tested on local

Types of changes (remove all unchecked types)

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Style (style only changes)
  • Docs
  • Refactor (code that does not add new functionality nor fixes a bug)

Checklist:

  • If my change requires a change to the documentation, I have updated the documentation accordingly, and in either case, have checked this box to attest to my assessment of this requirement with regard to my change.
  • If my change requires additions or updates to any deployment scripts to ensure that the protocol is functional (Makefile, Dockerfile, Forge Script, etc.), I have made these changes, and in either case, have checked this box to attest to my assessment of this requirement with regard to my change.
  • If my change requires additional test coverage, I have created those tests accordingly, and in either case, have checked this box to attest to my assessment of this requirement with regard to my change.

@kss-t1 kss-t1 requested a review from evchip January 9, 2026 14:43
@kss-t1 kss-t1 merged commit 5f8106b into canary Jan 9, 2026
4 checks passed
@kss-t1 kss-t1 deleted the feat/sealed-bid-auction/only-accept-prices-from-connected-solvers branch January 9, 2026 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants