Re: [pve-devel] [PATCH v6 proxmox-apt 04/11] add check_repositories function

2021-06-18 Thread Fabian Ebner
Am 18.06.21 um 09:16 schrieb Wolfgang Bumiller: On 06/18/2021 8:58 AM Fabian Ebner wrote: Am 18.06.21 um 08:56 schrieb Wolfgang Bumiller: On 06/18/2021 8:53 AM Fabian Ebner wrote: Am 18.06.21 um 08:44 schrieb Wolfgang Bumiller: On 06/18/2021 8:42 AM Fabian Ebner wrote: +

Re: [pve-devel] [PATCH v6 proxmox-apt 04/11] add check_repositories function

2021-06-18 Thread Wolfgang Bumiller
> On 06/18/2021 8:58 AM Fabian Ebner wrote: > > > Am 18.06.21 um 08:56 schrieb Wolfgang Bumiller: > > > >> On 06/18/2021 8:53 AM Fabian Ebner wrote: > >> > >> > >> Am 18.06.21 um 08:44 schrieb Wolfgang Bumiller: > >>> > On 06/18/2021 8:42 AM Fabian Ebner wrote: > >> +

Re: [pve-devel] [PATCH v6 proxmox-apt 04/11] add check_repositories function

2021-06-17 Thread Fabian Ebner
Am 18.06.21 um 08:56 schrieb Wolfgang Bumiller: On 06/18/2021 8:53 AM Fabian Ebner wrote: Am 18.06.21 um 08:44 schrieb Wolfgang Bumiller: On 06/18/2021 8:42 AM Fabian Ebner wrote: +Some((last, rest)) => match rest.split_last() { +Some((second_to_last, _rest

Re: [pve-devel] [PATCH v6 proxmox-apt 04/11] add check_repositories function

2021-06-17 Thread Wolfgang Bumiller
> On 06/18/2021 8:53 AM Fabian Ebner wrote: > > > Am 18.06.21 um 08:44 schrieb Wolfgang Bumiller: > > > >> On 06/18/2021 8:42 AM Fabian Ebner wrote: > +Some((last, rest)) => match rest.split_last() { > +Some((second_to_last, _rest)) => { > +

Re: [pve-devel] [PATCH v6 proxmox-apt 04/11] add check_repositories function

2021-06-17 Thread Fabian Ebner
Am 18.06.21 um 08:44 schrieb Wolfgang Bumiller: On 06/18/2021 8:42 AM Fabian Ebner wrote: +Some((last, rest)) => match rest.split_last() { +Some((second_to_last, _rest)) => { +(*last == "org" && *second_to_last == "debian") +

Re: [pve-devel] [PATCH v6 proxmox-apt 04/11] add check_repositories function

2021-06-17 Thread Wolfgang Bumiller
> On 06/18/2021 8:42 AM Fabian Ebner wrote: > >> +Some((last, rest)) => match rest.split_last() { > >> +Some((second_to_last, _rest)) => { > >> +(*last == "org" && *second_to_last == "debian") > >> +|| (*last == "com" && *se

Re: [pve-devel] [PATCH v6 proxmox-apt 04/11] add check_repositories function

2021-06-17 Thread Fabian Ebner
Am 17.06.21 um 10:39 schrieb Wolfgang Bumiller: some non-blocking cleanups in case you do another version: On Fri, Jun 11, 2021 at 01:43:53PM +0200, Fabian Ebner wrote: which checks for bad suites and official URIs. Signed-off-by: Fabian Ebner --- Changes from v5: * split out host_from_

Re: [pve-devel] [PATCH v6 proxmox-apt 04/11] add check_repositories function

2021-06-17 Thread Fabian Grünbichler
On June 11, 2021 1:43 pm, Fabian Ebner wrote: > which checks for bad suites and official URIs. > > Signed-off-by: Fabian Ebner > --- > > Changes from v5: > * split out host_from_uri helper and also handle userinfo and port > * test an offical URI with port > * match all *.debian.org

Re: [pve-devel] [PATCH v6 proxmox-apt 04/11] add check_repositories function

2021-06-17 Thread Wolfgang Bumiller
some non-blocking cleanups in case you do another version: On Fri, Jun 11, 2021 at 01:43:53PM +0200, Fabian Ebner wrote: > which checks for bad suites and official URIs. > > Signed-off-by: Fabian Ebner > --- > > Changes from v5: > * split out host_from_uri helper and also handle userinfo an

[pve-devel] [PATCH v6 proxmox-apt 04/11] add check_repositories function

2021-06-11 Thread Fabian Ebner
which checks for bad suites and official URIs. Signed-off-by: Fabian Ebner --- Changes from v5: * split out host_from_uri helper and also handle userinfo and port * test an offical URI with port * match all *.debian.org and *.proxmox.com as official to avoid (future) false nega