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

[Fabian Ebner]

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

> 
> _______________________________________________
> pbs-devel mailing list
> pbs-devel at lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel
> 
>