karaf git commit: [KARAF-4784] OsgiConfiguration for JAAS should fallback to default configuration

2016-12-01 Thread gnodet
Repository: karaf
Updated Branches:
  refs/heads/master 040c805f4 -> ad3e4bd98


[KARAF-4784] OsgiConfiguration for JAAS should fallback to default configuration

Project: http://git-wip-us.apache.org/repos/asf/karaf/repo
Commit: http://git-wip-us.apache.org/repos/asf/karaf/commit/ad3e4bd9
Tree: http://git-wip-us.apache.org/repos/asf/karaf/tree/ad3e4bd9
Diff: http://git-wip-us.apache.org/repos/asf/karaf/diff/ad3e4bd9

Branch: refs/heads/master
Commit: ad3e4bd98e71eba03fd79f8fae1194f4c803c7aa
Parents: 040c805
Author: Guillaume Nodet 
Authored: Thu Dec 1 17:16:03 2016 +0100
Committer: Guillaume Nodet 
Committed: Thu Dec 1 17:16:03 2016 +0100

--
 .../org/apache/karaf/jaas/config/impl/OsgiConfiguration.java | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
--


http://git-wip-us.apache.org/repos/asf/karaf/blob/ad3e4bd9/jaas/config/src/main/java/org/apache/karaf/jaas/config/impl/OsgiConfiguration.java
--
diff --git 
a/jaas/config/src/main/java/org/apache/karaf/jaas/config/impl/OsgiConfiguration.java
 
b/jaas/config/src/main/java/org/apache/karaf/jaas/config/impl/OsgiConfiguration.java
index 522507d..cdc2471 100644
--- 
a/jaas/config/src/main/java/org/apache/karaf/jaas/config/impl/OsgiConfiguration.java
+++ 
b/jaas/config/src/main/java/org/apache/karaf/jaas/config/impl/OsgiConfiguration.java
@@ -75,6 +75,8 @@ public class OsgiConfiguration extends Configuration {
 }
 
 public void refresh() {
-// Nothing to do, as we auto-update the configuration
+if (defaultConfiguration != null) {
+defaultConfiguration.refresh();
+}
 }
 }



karaf git commit: [KARAF-4784] OsgiConfiguration for JAAS should fallback to default configuration

2016-12-01 Thread gnodet
Repository: karaf
Updated Branches:
  refs/heads/master 7eea1ff73 -> 040c805f4


[KARAF-4784] OsgiConfiguration for JAAS should fallback to default configuration

Project: http://git-wip-us.apache.org/repos/asf/karaf/repo
Commit: http://git-wip-us.apache.org/repos/asf/karaf/commit/040c805f
Tree: http://git-wip-us.apache.org/repos/asf/karaf/tree/040c805f
Diff: http://git-wip-us.apache.org/repos/asf/karaf/diff/040c805f

Branch: refs/heads/master
Commit: 040c805f47abe0f0a3fd18258e14bec5f4bca5a8
Parents: 7eea1ff
Author: Guillaume Nodet 
Authored: Thu Dec 1 17:14:00 2016 +0100
Committer: Guillaume Nodet 
Committed: Thu Dec 1 17:14:36 2016 +0100

--
 .../karaf/jaas/config/impl/OsgiConfiguration.java   | 12 +++-
 1 file changed, 11 insertions(+), 1 deletion(-)
--


http://git-wip-us.apache.org/repos/asf/karaf/blob/040c805f/jaas/config/src/main/java/org/apache/karaf/jaas/config/impl/OsgiConfiguration.java
--
diff --git 
a/jaas/config/src/main/java/org/apache/karaf/jaas/config/impl/OsgiConfiguration.java
 
b/jaas/config/src/main/java/org/apache/karaf/jaas/config/impl/OsgiConfiguration.java
index 6c995ba..522507d 100644
--- 
a/jaas/config/src/main/java/org/apache/karaf/jaas/config/impl/OsgiConfiguration.java
+++ 
b/jaas/config/src/main/java/org/apache/karaf/jaas/config/impl/OsgiConfiguration.java
@@ -23,18 +23,26 @@ import javax.security.auth.login.Configuration;
 
 import org.apache.karaf.jaas.config.JaasRealm;
 import org.apache.karaf.util.collections.CopyOnWriteArrayIdentityList;
+import org.slf4j.LoggerFactory;
 
 public class OsgiConfiguration extends Configuration {
 
 private final List realms = new 
CopyOnWriteArrayIdentityList();
+private Configuration defaultConfiguration;
 
 public void init() {
+try {
+defaultConfiguration = Configuration.getConfiguration();
+} catch (Throwable ex) {
+// default configuration for fallback could not be retrieved
+LoggerFactory.getLogger(OsgiConfiguration.class).warn("Unable to 
retrieve default configuration", ex);
+}
 Configuration.setConfiguration(this);
 }
 
 public void close() {
 realms.clear();
-Configuration.setConfiguration(null);
+Configuration.setConfiguration(defaultConfiguration);
 }
 
 public void register(JaasRealm realm, Map properties) {
@@ -60,6 +68,8 @@ public class OsgiConfiguration extends Configuration {
 }
 if (realm != null) {
 return realm.getEntries();
+} else if (defaultConfiguration != null) {
+return defaultConfiguration.getAppConfigurationEntry(name);
 }
 return null;
 }