shinesolutions / ruby_aem

Ruby client for Adobe Experience Manager (AEM) API
Apache License 2.0
13 stars 7 forks source link

Support org.apache.felix.proxy.load.balancer.connection.enable property for org.apache.felix.http #32

Closed henrykuijpers closed 4 years ago

henrykuijpers commented 4 years ago
      # As written in the SSL offloading documentation on the Adobe site
      # https://helpx.adobe.com/experience-manager/kb/AEM-redirecting-back-to-http-on-accessed-via-SSL-terminated-Load-Balancer.html#main-pars_text_1810611796
      aem_config_property {"[${instance_config['aem_id']}] Create org.apache.felix.proxy.load.balancer.connection.enable property":
        ensure           => present,
        name             => 'org.apache.felix.proxy.load.balancer.connection.enable',
        type             => 'Boolean',
        value            => true,
        run_mode         => $instance_config['aem_id'],
        config_node_name => 'org.apache.felix.http',
        aem_id           => $instance_config['aem_id'],
        require          => Aem_Aem["[${instance_config['aem_id']}] Wait until org.apache.felix.http OSGI config is set"],
      } -> exec { "[${instance_config['aem_id']}] Wait org.apache.felix.proxy.load.balancer.connection.enable property":
        command => 'sleep 5',
        path    => ['/usr/bin', '/usr/sbin', '/bin'],
      } -> aem_aem { "[${instance_config['aem_id']}] Wait until aem health check is ok org.apache.felix.proxy.load.balancer.connection.enable property":
        ensure       => aem_health_check_is_ok,
        tags         => 'deep',
        aem_id       => $instance_config['aem_id'],
      } -> aem_aem { "[${instance_config['aem_id']}] Wait until org.apache.felix.proxy.load.balancer.connection.enable property is set":
        ensure       => login_page_is_ready,
        aem_id       => $instance_config['aem_id'],
      }

It seems this property is not honored, since it is not in the spec. I would expect this to throw an error or something like that.

mbloch1986 commented 4 years ago

hi @henrykuijpers is this related to #31 ?

henrykuijpers commented 4 years ago

Yes, I think so!

mbloch1986 commented 4 years ago

@henrykuijpers Thanks... I would close this ticket as #31 talks about the same problem.