Skip to content

Look into cleaning up multi-protocol support #1603

@sisuresh

Description

@sisuresh

At the moment, the code has support for setting a range of protocol versions supported by env (example). This was done back when we required backwards compatibility with previous protocols, but now we just package a new env per protocol release, and no longer require backwards compatibility.

There are a couple things we should look into removing -

  1. The TEST_PROTOCOL environment variable and any related code.
  2. Code used by the "next" feature. I don't think we can remove the feature itself because it's used to activate xdr staged for the next protocol.

Before we do any of this, we should make sure we're fine with removing multi-protocol support in env. I think it's fine to remove, but I believe we've discussed using this functionality for small protocol changes to reduce the number of hosts in core. That doesn't sound like a good idea due to complexity, but wanted to point it out.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions