[ 
https://issues.apache.org/jira/browse/SLING-5752?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15306771#comment-15306771
 ] 

Timothee Maret commented on SLING-5752:
---------------------------------------

The issue is fixed in the PR https://github.com/apache/sling/pull/144 (patch at 
https://patch-diff.githubusercontent.com/raw/apache/sling/pull/144.patch).

This patch has been tested successfully with against a Sling instance running a 
SSL connector.
I would add integration tests in 
{{o.a.s.d.t.i.SimpleHttpDistributionTransportTest}} but it seems the test case 
is ignored.

[~teofili], [~mpetria] any idea why the tests are ignored ? Could you merge 
this patch ?

> SimpleHttpDistributionTransport does not leverage configured scheme from 
> distribution endpoint
> ----------------------------------------------------------------------------------------------
>
>                 Key: SLING-5752
>                 URL: https://issues.apache.org/jira/browse/SLING-5752
>             Project: Sling
>          Issue Type: Bug
>          Components: Distribution, Extensions
>    Affects Versions: Content Distribution Core 0.1.16
>            Reporter: Timothee Maret
>            Assignee: Timothee Maret
>             Fix For: Content Distribution Core 0.1.18
>
>
> {{SimpleHttpDistributionTransport}} currently does not allow to communicate 
> with a distribution endpoint using {{https://}} scheme. The root cause is 
> that the method 
> {{org.apache.sling.distribution.transport.impl.SimpleHttpDistributionTransport#authenticate}}
>  does hardcode the scheme instead of setting it from the configured 
> {{distributionEndpoint}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to