xpublish-community / xpublish

Publish Xarray Datasets via a REST API.
https://xpublish.readthedocs.io
Apache License 2.0
168 stars 23 forks source link

Set zarr router prefix #182

Closed abkfenris closed 1 year ago

abkfenris commented 1 year ago

We should set a default prefix for the Zarr router plugin. Otherwise it often wants to eat the requests to other dataset routers.

Brought up by @kwilcox during the community meeting #172 and @jhamman gave a thumbs up to make the change.

I'll assign myself to make sure it gets done before the next release, but if someone else gets to it first, go for it.

It should be changing the prefix here:

https://github.com/xarray-contrib/xpublish/blob/feff2c44ad9e533adf41753c29a0c16544d2868c/xpublish/plugins/included/zarr.py#L21-L30

todataset_router_prefix: str = '/zarr'and then cleaning up the tests.