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

[Wolfgang Bumiller]

> On 06/18/2021 9:07 AM Fabian Ebner <f.ebner at proxmox.com> wrote:
> 
>  
> Am 18.06.21 um 08:58 schrieb Fabian Ebner:
> > Am 18.06.21 um 08:56 schrieb Wolfgang Bumiller:
> >>
> >>> On 06/18/2021 8:53 AM Fabian Ebner <f.ebner at proxmox.com> wrote:
> >>>
> >>> Am 18.06.21 um 08:44 schrieb Wolfgang Bumiller:
> >>>>
> >>>>> On 06/18/2021 8:42 AM Fabian Ebner <f.ebner at proxmox.com> wrote:
> >>>>>>> +            Some((last, rest)) => match rest.split_last() {
> >>>>>>> +                Some((second_to_last, _rest)) => {
> >>>>>>> +                    (*last == "org" && *second_to_last == "debian")
> >>>>>>> +                        || (*last == "com" && *second_to_last == 
> >>>>>>> "proxmox")
> >>>>>>> +                }
> >>>>>>> +                None => false,
> >>>>>>> +            },
> >>>>>>> +            None => false,
> >>>>>>> +        };
> >>>>>>> +
> >>>>>>> +        for uri in self.uris.iter() {
> >>>>>>> +            if let Some(host) = host_from_uri(uri) {
> >>>>>>> +                let domains = host.split('.').collect();
> >>>>>>
> >>>>>> ^ But instead of building a vector here, why not just do:
> >>>>>>
> >>>>>>        if host == "proxmox.com" || host.ends_with(".proxmox.com")
> >>>>>>            || host == "debian.org" || host.ends_with(".debian.org")
> >>>>>>        {
> >>>>>>            ...
> >>>>>>        }
> >>>>>>
> >>>>>
> >>>>> Misses FQDNs?
> >>>>
> >>>> Such as?
> >>>>
> >>>
> >>> http://security.debian.org.
> >>
> >> Why is that not caught by `.ends_with(".debian.org")`?
> >>
> > 
> > Because of the final dot. But it is likely very uncommon and simply 
> > splitting by '.' leads to false results with e.g. 
> > http://security..debian.org too, so it might not be worth worrying about...
> > 
> 
> Sorry, should've been http://security.debian..org

I still can't navigate to that in my browser ;-)