[pbs-devel] applied: [PATCH proxmox v3 1/2] metrics: encode influxdb org and bucket parameters
Thomas Lamprecht
t.lamprecht at proxmox.com
Fri Apr 26 17:52:13 CEST 2024
Am 26/04/2024 um 16:02 schrieb Gabriel Goller:
> In order to remove the current limitations on the bucket and
> organization names, we need to make sure that they are transmitted
> correctly. In order to do this, we encode them using the url crate.
>
> This way we support organization/bucket names that include slashes,
> whitespaces, etc.
>
> Signed-off-by: Gabriel Goller <g.goller at proxmox.com>
> ---
> proxmox-metrics/Cargo.toml | 1 +
> proxmox-metrics/src/influxdb/http.rs | 7 ++++++-
> 2 files changed, 7 insertions(+), 1 deletion(-)
>
>
applied, thanks!
Having some tests here might be nice though. E.g., moving the URI assembly
into a separate method and having either a test module or doc-tests to
ensure that some common and odd cases, e.g. those requiring encoding, are
handled correctly.
More information about the pbs-devel
mailing list