Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ovsx-client: support multiple registries #12040

Merged
merged 1 commit into from
Jun 15, 2023
Merged

Commits on Jun 15, 2023

  1. ovsx-client: support multiple registries

    Refactors the `OVSXClient` into a simple interface that can easily be
    implemented by components such as `OVSXHttpClient` or others.
    
    Add a new `OVSXRouterClient` that may route OpenVSX queries to different
    registries based on an `OVSXRouteConfig` configuration. The path to the
    configuration file can be specified when starting the backend using the
    `--ovsx-router-config` CLI param. When this configuration is not
    specified, we default to the previously configured OpenVSX registry.
    
    The goal of these changes was to minimally impact Theia's plugin code
    base by simply replacing the client interfaces we use to communicate
    with an OpenVSX registry. This proved more difficult than expected,
    especially when merging search results: pagination may currently be
    broken when dealing with multiple registries.
    
    Add a sample mock OpenVSX server running as part of Theia's backend.
    You can find the plugins served by this mock server under
    `examples/api-samples/plugins-db`. This mock is there to allow
    easier testing of the multi-registry feature.
    paul-marechal committed Jun 15, 2023
    Configuration menu
    Copy the full SHA
    7fea930 View commit details
    Browse the repository at this point in the history