eclipse-threadx / netxduo

Eclipse ThreadX - NetXDuo is an advanced, industrial-grade TCP/IP network stack designed specifically for deeply embedded real-time and IoT applications
https://github.com/eclipse-threadx/rtos-docs/blob/main/rtos-docs/netx-duo/index.md
MIT License
247 stars 140 forks source link

pop3 client port number #21

Closed ReKawashima closed 3 years ago

ReKawashima commented 3 years ago

Why does pop3 client use a specific port "4228"?

https://github.com/azure-rtos/netxduo/blob/617d1f0bb79ed1623efdeebabb100e299ef266ce/addons/pop3/nxd_pop3_client.c#L2680

bo-ms commented 3 years ago

Hi @ReKawashima, thanks for bringing it up, this bug will be fixed in next release. Closing the issue now. Feel free to re-open if you have additional questions.

ReKawashima commented 3 years ago

Hi @bo-ms -san, Thank you for response and confirming this bug. I want to know your solution for this bug. Change to "NX_ANY_PORT"?

And, NetX POP3 Client has same bug. Please fix NetX POP3 Client as well. https://github.com/azure-rtos/netx/blob/490762909bbdb52250c1b491324ba601359ee6ca/addons/pop3/nx_pop3_client.c#L2468

bo-ms commented 3 years ago

Hi @ReKawashima You are correct, changed it to NX_ANY_PORT. Thanks for reminding, I already fixed both files.