intuit / QuickBooks-V3-Java-SDK

Java SDK for QuickBooks REST API v3 services
Apache License 2.0
66 stars 144 forks source link

Url's port ignored when Apache HTTP Client is used #182

Open sukolenvo opened 2 years ago

sukolenvo commented 2 years ago

Default http transport is Apache HTTP Client (link). But it doesn't support url port (ie. I can't use http://locahost:8080/ as BASE_URL_QBO in tests. Requests will be made to localhost:80).

I think root cause is that -1 is used as port in HttpTarget (source) Is it possible to replace hardcoded value with url.getPort() (if port ommitted - it returns -1 anyway)? Happy to open PR.