Skip to content

Conversation

yashvardhan-kukreja
Copy link
Contributor

@yashvardhan-kukreja yashvardhan-kukreja commented Jul 9, 2025

Signed-off-by: Yashvardhan Kukreja [email protected]

Description

This PR:

  • adds support for forceDisableInteropValidation - an easy-to-manage toggle to reject all interop transactions (useful in a scenario where we want to prevent any communication with the backing supervisors in case they're in maintenance)
  • Cleans up all the rpc codes related to interop. Previously that was a leaky abstraction supposed to be completely served by op-supervisor but due to the fact it wasn't, proxyd had to compensate for it. Now, with this PR, proxyd can just inherit the rpc error code directly coming from supervisor's response.
  • [In progress] Auto-stop backoff: backoff from making calls to supervisor backend for a finite amount of time when receiving backoff errors.

Tests

Please describe any tests you've added. If you've added no tests, or left important behavior untested, please explain why not.

Additional context

Add any other context about the problem you're solving.

Metadata

  • Fixes #[Link to Issue]

@yashvardhan-kukreja yashvardhan-kukreja requested a review from a team as a code owner July 9, 2025 23:44
Copy link

wiz-inc-a178a98b5d bot commented Jul 9, 2025

Wiz Scan Summary

Scanner Findings
Vulnerability Finding Vulnerabilities 1 Medium
Data Finding Sensitive Data -
Secret Finding Secrets -
IaC Misconfiguration IaC Misconfigurations -
Total 1 Medium

View scan details in Wiz

To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension.

@yashvardhan-kukreja yashvardhan-kukreja marked this pull request as draft July 9, 2025 23:45
@yashvardhan-kukreja yashvardhan-kukreja changed the title feat: force disable interop validation and remove interop error code awareness feat: force disable interop validation toggle and remove interop error code awareness Jul 9, 2025
Signed-off-by: Yashvardhan Kukreja <[email protected]>
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.

1 participant