From 54144b697bd1a5c4b08e8da9fec655afd949446c Mon Sep 17 00:00:00 2001 From: Thomas Lenz Date: Mon, 26 May 2014 14:47:36 +0200 Subject: fix different STORK bugs --- .../moa/id/configuration/data/oa/OASTORKConfig.java | 12 +++++++++--- .../id/configuration/struts/action/EditOAAction.java | 18 +++++++++++++----- .../validation/oa/OAPVP2ConfigValidation.java | 4 ++-- 3 files changed, 24 insertions(+), 10 deletions(-) (limited to 'id/ConfigWebTool/src/main/java') diff --git a/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/data/oa/OASTORKConfig.java b/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/data/oa/OASTORKConfig.java index be5b2200b..363c094ff 100644 --- a/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/data/oa/OASTORKConfig.java +++ b/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/data/oa/OASTORKConfig.java @@ -253,9 +253,15 @@ public class OASTORKConfig implements IOnlineApplicationData{ public List getEnabledCPEPS() { if (enabledCitizenCountries != null) { List result = new ArrayList(); - for(CPEPS current : ConfigurationDBRead.getMOAIDConfiguration().getAuthComponentGeneral().getForeignIdentities().getSTORK().getCPEPS()) { - if(enabledCitizenCountries.contains(current.getCountryCode())) - result.add(current); + + try { + for(CPEPS current : ConfigurationDBRead.getMOAIDConfiguration().getAuthComponentGeneral().getForeignIdentities().getSTORK().getCPEPS()) { + if(enabledCitizenCountries.contains(current.getCountryCode())) + result.add(current); + } + + } catch (NullPointerException e){ + } return result; } diff --git a/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/struts/action/EditOAAction.java b/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/struts/action/EditOAAction.java index 659835d03..9509f9712 100644 --- a/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/struts/action/EditOAAction.java +++ b/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/struts/action/EditOAAction.java @@ -133,12 +133,20 @@ public class EditOAAction extends BasicOAAction { // prepare attribute helper list ArrayList attributes = new ArrayList(); - for (StorkAttribute current : ConfigurationDBRead - .getMOAIDConfiguration().getAuthComponentGeneral() - .getForeignIdentities().getSTORK().getAttributes()) - attributes.add(new AttributeHelper(current)); - getStorkOA().setHelperAttributes(attributes); + + try { + for (StorkAttribute current : ConfigurationDBRead + .getMOAIDConfiguration().getAuthComponentGeneral() + .getForeignIdentities().getSTORK().getAttributes()) + attributes.add(new AttributeHelper(current)); + + + } catch (NullPointerException e) { + + } + getStorkOA().setHelperAttributes(attributes); + } catch (BasicActionException e) { return Constants.STRUTS_ERROR; diff --git a/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/validation/oa/OAPVP2ConfigValidation.java b/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/validation/oa/OAPVP2ConfigValidation.java index 345d25cda..40e243d0b 100644 --- a/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/validation/oa/OAPVP2ConfigValidation.java +++ b/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/validation/oa/OAPVP2ConfigValidation.java @@ -121,10 +121,10 @@ public class OAPVP2ConfigValidation { protoSocketFactory); } catch (MOAHttpProtocolSocketFactoryException e) { - log.warn("MOA SSL-TrustStore can not initialized. Use default Java TrustStore."); + log.warn("MOA SSL-TrustStore can not initialized. Use default Java TrustStore.", e); } catch (ConfigurationException e) { - log.info("No MOA specific SSL-TrustStore configured. Use default Java TrustStore."); + log.info("No MOA specific SSL-TrustStore configured. Use default Java TrustStore.", e); } -- cgit v1.2.3