Skip to content

proxyd: Handle unexpected JSON-RPC responses#2628

Merged
mergify[bot] merged 2 commits intoethereum-optimism:developfrom
Inphi:inphi/provider-causality
Jun 8, 2022
Merged

proxyd: Handle unexpected JSON-RPC responses#2628
mergify[bot] merged 2 commits intoethereum-optimism:developfrom
Inphi:inphi/provider-causality

Conversation

@Inphi
Copy link
Contributor

@Inphi Inphi commented May 27, 2022

This fixes a bug where the infura backend would be labeled offline because it
returns an unexpected JSON-RPC response. Unexpected, but well-formed,
JSON-RPC response are handled specially. Such errors are surfaced up to
the backend proxier so failover still occurs.

This fixes a bug where the infura backend would be labeled offline because it
returns an unexpected JSON-RPC response. Unexpected, but well-formed,
JSON-RPC response are handled specially. Such errors are surfaced up to
the backend proxier so failover still occurs.
@changeset-bot
Copy link

changeset-bot bot commented May 27, 2022

🦋 Changeset detected

Latest commit: 5713ee3

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@eth-optimism/proxyd Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@mergify mergify bot requested review from mslipper and tuxcanfly May 27, 2022 19:09
@mergify
Copy link
Contributor

mergify bot commented Jun 8, 2022

This PR has been added to the merge queue, and will be merged soon.

@mergify
Copy link
Contributor

mergify bot commented Jun 8, 2022

This PR is next in line to be merged, and will be merged as soon as checks pass.

@mergify mergify bot merged commit 58dc7ad into ethereum-optimism:develop Jun 8, 2022
@mergify mergify bot removed the on-merge-train label Jun 8, 2022
This was referenced Jun 8, 2022
theochap pushed a commit that referenced this pull request Dec 10, 2025
Fixes #2338

---------

Co-authored-by: Arun Dhyani <dhyaniarun7@gmail.com>
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