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

[Jon Humphreys <j-humphreys@ti.com>]

Jerome Forissier <jerome.forissier@linaro.org> writes:

> 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?
>

No problem.

Jon

> 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
>>>