Re: [PATCH v11 20/29] net-lwip: lwIP wget supports user defined port in the uri, so allow it.

[Jerome Forissier <jerome.forissier@linaro.org>]

On 10/4/24 09:11, Ilias Apalodimas wrote:
> On Thu, 3 Oct 2024 at 18:47, Jerome Forissier
> <jerome.forissier@linaro.org> wrote:
>>
>> From: Jonathan Humphreys <j-humphreys@ti.com>
> 
> A short description is missing here

I will fold this patch into the previous one since it was a review
comment from Jonathan H. ; Jonathan I hope you don't mind?

Thanks,
-- 
Jerome

> 
> Thanks
> /Ilias
>>
>> Signed-off-by: Jonathan Humphreys <j-humphreys@ti.com>
>> Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
>> ---
>>  net/lwip/wget.c | 6 ------
>>  1 file changed, 6 deletions(-)
>>
>> diff --git a/net/lwip/wget.c b/net/lwip/wget.c
>> index e8768fc2acd..b495ebd1aa9 100644
>> --- a/net/lwip/wget.c
>> +++ b/net/lwip/wget.c
>> @@ -349,12 +349,6 @@ bool wget_validate_uri(char *uri)
>>                 ret = false;
>>                 goto out;
>>         }
>> -       s = strchr(authority, ':');
>> -       if (s) {
>> -               log_err("user defined port is not supported\n");
>> -               ret = false;
>> -               goto out;
>> -       }
>>
>>  out:
>>         free(str_copy);
>> --
>> 2.40.1
>>