[pbs-devel] [PATCH backup] client: use build_authority in build_uri

Wolfgang Bumiller w.bumiller at proxmox.com
Thu May 6 09:43:31 CEST 2021


On Thu, May 06, 2021 at 09:34:03AM +0200, Dietmar Maurer wrote:
> > On 05/06/2021 9:14 AM Wolfgang Bumiller <w.bumiller at proxmox.com> wrote:
> > 
> >  
> > On Thu, May 06, 2021 at 09:01:46AM +0200, Dietmar Maurer wrote:
> > > In general a good idea, but we now merge code
> > > from two independent http client implementation (making them
> > > dependent)?
> > 
> > I mean... it's called "tools::http"... as in "http utilities", which
> > IMO does not really sound like "this is an independent http client,
> > which must not be reused"... ;-)
> 
> What naming do you suggest instead?

I suppose I'd put shared http stuff in `tools::http` and if the client
should be separate I'd suggest `tools::http::client`, but I'm not
convinced it cannot stay as is while sharing some code in the first
place

> 
> > And it's a rather thin layer around hyper (the *actual* client...) to
> > add proxy support which, too, is something that we may at some point
> > want to reuse without the rest of `SimpleHttp`, no?
> 
> I am not sure if it is worth to reuse that for that backup client. We simply
> do not need proxy support there.

yet ;-)





More information about the pbs-devel mailing list