I guess that would have to go into the 1.13 branch as well. This changes the public API but I think we should do it. The current API isn't usable since it refers to a non-public interface.
On 5/4/20, 9:31 AM, "Mario Ivanac" <mario.iva...@est.tech> wrote: Hi all, after comments that SSLParameterExtension interface has an init() method that takes a DistributionConfig as an argument (which is internal class), new solution is proposed to replace DistributionConfig with Properties. New PR is created with new proposal https://github.com/apache/geode/pull/5040, and also RFC is updated with new proposal https://cwiki.apache.org/confluence/display/GEODE/Introduction+of+SSL+Parameter+Extension Introduction of SSL Parameter Extension - Geode - Apache Software Foundation<https://cwiki.apache.org/confluence/display/GEODE/Introduction+of+SSL+Parameter+Extension> Hit enter to search. Help. Online Help Keyboard Shortcuts Feed Builder What’s new cwiki.apache.org Should we cherry-picked this PR into the support/1.12 branch? Regards, Mario