Don't assume the request URL is on port 80

Zolli / Phabricator-PHP-API

PHP API for Phabricator Conduit application

GNU Lesser General Public License v3.0

21 stars 5 forks source link

Don't assume the request URL is on port 80 #1

Closed brianherbert closed 9 years ago

brianherbert commented 9 years ago

I had an issue where I couldn't call an API since it had SSL enabled. Removing the static port setting in cURL fixed this since SSL defaults to port 443.

Apologies for not doing this through your Phabricator setup. I have to wait to get my Phab account approved so I'm doing this here. Feel free to close this out.

Zolli commented 9 years ago

Hi,

nice fix, i merge it. Currently, M phabricator is not under use, because i focus my energy to my company phabricator instance. I approve Your account when the configuration is fully done (Needs prorper permissions)

Thanks.

brianherbert commented 9 years ago

No problem, @Zolli - Thanks for the merge!

Zolli commented 9 years ago

Okay, i tidyed my instance a bit and set-up mirroring from phabricator to here. Also approved your user, feel free to submit patches, or improvements on the future.