search

PredixDev / predix-rest-client

Other
7 stars 16 forks source link

Incorrect assignment in DefaultOauthRestConfig.setOauthProxyHost #10

Open davenunn opened 6 years ago

davenunn commented 6 years ago

Within DefaultOauthRestConfig.setOauthProxyHost:

If oauthApplyProxyPropertiesToSystemProperties is true then the following code is executed (lines 211 and 212):

System.setProperty("https.proxyHost", this.oauthProxyHost); System.setProperty("http.proxyPort", this.oauthProxyHost);

The second line will set http.proxyPort to the proxy host. This should read:

System.setProperty("http.proxyHost", this.oauthProxyHost);

gstroup commented 6 years ago

Thanks for pointing this out! We'll push out the fix soon.