Skip to content

Commit

Permalink
Renames container_security_options to container_security_options
Browse files Browse the repository at this point in the history
  • Loading branch information
josejulio committed Mar 2, 2017
1 parent 4617376 commit dd82a94
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions app/controllers/mixins/ems_common_angular.rb
Original file line number Diff line number Diff line change
Expand Up @@ -424,19 +424,19 @@ def set_ems_record_vars(ems, mode = nil)
if ems.kind_of?(ManageIQ::Providers::ContainerManager)
params[:cred_type] = ems.default_authentication_type if params[:cred_type] == "default"
default_endpoint = {:role => :default, :hostname => hostname, :port => port}
default_endpoint.merge!(container_security_options(ems.security_protocol, default_tls_ca_certs))
default_endpoint.merge!(endpoint_security_options(ems.security_protocol, default_tls_ca_certs))

if hawkular_hostname.blank?
default_key = params[:default_password] || ems.authentication_key
hawkular_hostname = get_hostname_from_routes(ems, default_endpoint, default_key)
end
hawkular_endpoint = {:role => :hawkular, :hostname => hawkular_hostname, :port => hawkular_api_port}
hawkular_endpoint.merge!(container_security_options(hawkular_security_protocol, hawkular_tls_ca_certs))
hawkular_endpoint.merge!(endpoint_security_options(hawkular_security_protocol, hawkular_tls_ca_certs))
end

if ems.kind_of?(ManageIQ::Providers::MiddlewareManager)
default_endpoint = {:role => :default, :hostname => hostname, :port => port}
default_endpoint.merge!(container_security_options(ems.security_protocol, default_tls_ca_certs))
default_endpoint.merge!(endpoint_security_options(ems.security_protocol, default_tls_ca_certs))
end

if ems.kind_of?(ManageIQ::Providers::Hawkular::DatawarehouseManager)
Expand Down Expand Up @@ -473,7 +473,7 @@ def get_hostname_from_routes(ems, endpoint_hash, token)
nil
end

def container_security_options(security_protocol, certificate_authority)
def endpoint_security_options(security_protocol, certificate_authority)
{
:security_protocol => security_protocol,
:verify_ssl => %w(ssl-without-validation non-ssl).exclude?(security_protocol),
Expand Down

0 comments on commit dd82a94

Please sign in to comment.