Skip to content
This repository has been archived by the owner on Nov 9, 2022. It is now read-only.

Unable to deploy modules if XCC port is secured with TLS #862

Open
mwarnes opened this issue Nov 23, 2017 · 4 comments
Open

Unable to deploy modules if XCC port is secured with TLS #862

mwarnes opened this issue Nov 23, 2017 · 4 comments

Comments

@mwarnes
Copy link

mwarnes commented Nov 23, 2017

RFE

Roxy is unable to deploy modules if he XCC port is secure using TLS

Within ./deploy/lib/xcc.rb the url is hardcoded to use http.

  def build_load_uri(target_uri, options, commit)
  url = "http://#{@hostname}:#{@port}/insert?"

It would be useful if the url was created with the protocol set to http our https depending on a configuration setting such as in ./deploy/lib/ml_rest.rb, i.e

 @rest_protocol = "http#{options[:use_https_for_rest] ? 's' : ''}"
 ...
 url = "#{@rest_protocol}://#{@hostname}:#{@port}/v1/config/properties"

The issue can be worked round by hardcoding https in the xcc.rb url.

@grtjn
Copy link
Contributor

grtjn commented Nov 23, 2017

I'm surprised this hasn't surfaced before yet. Shouldn't be too difficult to implement. Would you be willing to fork the project, and open a PR? You can find guidelines in CONTRIBUTING.md..

@heelix
Copy link

heelix commented Feb 8, 2018

Do you know if this one was merged in? We are trying to add in a few SSL xcc ports to our app via ROXY (rather than tweak by hand). I can pick up this ball if it did not go anywhere.

@grtjn
Copy link
Contributor

grtjn commented Feb 13, 2018

I don't think anyone looked into this yet, so feel free to open a PR!

@heelix
Copy link

heelix commented Feb 22, 2018

Will do. Planning to get Roxy current in our environment as soon as 9.0-4.2 hits the downloads. Will be testing on that.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants