Adobe-Consulting-Services / acs-aem-commons

http://adobe-consulting-services.github.io/acs-aem-commons/
Apache License 2.0
454 stars 600 forks source link

Remove mandatory limitation for port of Http Client Factory feature in ACS AEM Commons #3386

Open leadstec-fltang opened 3 months ago

leadstec-fltang commented 3 months ago

Hello guys,

Http Client Factory feature is a good feature. But I find that we must config the port as saying in this document: https://adobe-consulting-services.github.io/acs-aem-commons/features/http-client-factory/index.html

But sometimes, we cannot using port. We want to leave it empty. Can you guys remove the mandatory limitation for port?

Thanks.

Best Regards

kwin commented 1 month ago

A port is mandatory for IP based connections, but it is often left out if the default for a specific protocol is used (80 for HTTP or 443 for HTTPS). As the protocol is not given here the port is necessary!