Skip to content

Conversation

@evchip
Copy link
Contributor

@evchip evchip commented Dec 26, 2025

Description

  • Add a polling fallback for Open intent events if websocket connection fails, with some logic to prevent processing duplicate events.
  • add pagerduty notifier for failed websocket

Motivation and Context

We believe the reason for some intents not being filled is the sealed bid auction's websocket connection to pick up Open intent events from the RPC provider is failing. This adds logic to still pick up the events even if the websocket fails

How Has This Been Tested?

Not tested. Will deploy and test.

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.

@evchip evchip changed the title Refactor/send open event log to filler Fix: polling fallback on failed websocket connection Dec 29, 2025
@evchip evchip requested a review from kss-t1 December 29, 2025 15:40
@evchip evchip self-assigned this Dec 29, 2025
@evchip evchip marked this pull request as ready for review December 29, 2025 15:41
Copy link
Member

@kss-t1 kss-t1 left a comment

Choose a reason for hiding this comment

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

This has conflicts with the solution for WS outage resistance. Please merge in upstream into this branch.

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.

3 participants