[ https://issues.apache.org/jira/browse/KARAF-4567?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Paolo Antinori updated KARAF-4567: ---------------------------------- Description: I'm trying to make support for encrypted properties in in ConfigAdmin and I cannot make it. My suspect is that the functionality is not really working and that the unit test is indeed not testing the functionality. The test is https://github.com/apache/karaf/blob/master/jaas/blueprint/jasypt/src/test/java/org/apache/karaf/jaas/blueprint/jasypt/handler/EncryptableConfigAdminPropertyPlaceholderTest.java and I think it should not assert for the correct value of {{foo}} but it should check the correct value of {{encoded}}, coming from https://github.com/apache/karaf/blob/master/jaas/blueprint/jasypt/src/test/resources/org/apache/karaf/jaas/blueprint/jasypt/handler/configadmin-test.xml I'm still on this and I might be wrong, but for what I can see, tokens like {{ENC($\{prop})}} fail to be replaced because {{org.apache.karaf.jaas.jasypt.handler.EncryptablePropertyPlaceholder}} always try to encrypt strings like {{$\{prop\}}} that have not been correctly replaced. was: I'm trying to make support for encrypted properties in in ConfigAdmin and I cannot make it. My suspect is that the functionality is not really working and that the unit test is indeed not testing the functionality. The test is https://github.com/apache/karaf/blob/master/jaas/blueprint/jasypt/src/test/java/org/apache/karaf/jaas/blueprint/jasypt/handler/EncryptableConfigAdminPropertyPlaceholderTest.java and I think it should not assert for the correct value of {{foo}} but it should check the correct value of {{encoded}}, coming from https://github.com/apache/karaf/blob/master/jaas/blueprint/jasypt/src/test/resources/org/apache/karaf/jaas/blueprint/jasypt/handler/configadmin-test.xml I'm still on this and I might be wrong, but for what I can see, tokens like {{ENC(${prop})}} fail to be replaced because {{org.apache.karaf.jaas.jasypt.handler.EncryptablePropertyPlaceholder}} always try to encrypt strings like {{${prop}}} that have not been correctly replaced. > ENC( not working in ConfigAdmin > -------------------------------- > > Key: KARAF-4567 > URL: https://issues.apache.org/jira/browse/KARAF-4567 > Project: Karaf > Issue Type: Bug > Affects Versions: 2.4.4, 4.0.5 > Reporter: Paolo Antinori > > I'm trying to make support for encrypted properties in in ConfigAdmin and I > cannot make it. > My suspect is that the functionality is not really working and that the unit > test is indeed not testing the functionality. > The test is > https://github.com/apache/karaf/blob/master/jaas/blueprint/jasypt/src/test/java/org/apache/karaf/jaas/blueprint/jasypt/handler/EncryptableConfigAdminPropertyPlaceholderTest.java > and I think it should not assert for the correct value of {{foo}} but it > should check the correct value of {{encoded}}, coming from > https://github.com/apache/karaf/blob/master/jaas/blueprint/jasypt/src/test/resources/org/apache/karaf/jaas/blueprint/jasypt/handler/configadmin-test.xml > I'm still on this and I might be wrong, but for what I can see, tokens like > {{ENC($\{prop})}} fail to be replaced because > {{org.apache.karaf.jaas.jasypt.handler.EncryptablePropertyPlaceholder}} > always try to encrypt strings like {{$\{prop\}}} that have not been correctly > replaced. -- This message was sent by Atlassian JIRA (v6.3.4#6332)