Skip to content

chore: extra logging on error from viem#198

Merged
kss-t1 merged 1 commit intocanaryfrom
chore/sealed-bid-auction/intent-watching-error-log
Sep 25, 2025
Merged

chore: extra logging on error from viem#198
kss-t1 merged 1 commit intocanaryfrom
chore/sealed-bid-auction/intent-watching-error-log

Conversation

@kss-t1
Copy link
Copy Markdown
Member

@kss-t1 kss-t1 commented Sep 25, 2025

Description

Simple log for edge case we encountered when Tokka disconnected due to exception on their side. It will give us more info, should similar situation ever happened again.

Motivation and Context

How Has This Been Tested?

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 rymcol September 25, 2025 15:23
@kss-t1 kss-t1 merged commit ac57527 into canary Sep 25, 2025
4 checks passed
@kss-t1 kss-t1 deleted the chore/sealed-bid-auction/intent-watching-error-log branch September 25, 2025 15:29
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