diff --git a/lib/SimpleSAML/Metadata/SAMLParser.php b/lib/SimpleSAML/Metadata/SAMLParser.php index 216b75784cac1c76aecb7c76af29d76b623f06bd..b8f082c116b5151b38358082001e7bea61021ff2 100644 --- a/lib/SimpleSAML/Metadata/SAMLParser.php +++ b/lib/SimpleSAML/Metadata/SAMLParser.php @@ -420,9 +420,14 @@ class SimpleSAML_Metadata_SAMLParser { /* * Add organizational metadata */ - $ret['name'] = $this->organizationDisplayName; - $ret['description'] = $this->organizationName; - + if (!empty($this->organizationName)) { + $ret['name'] = $this->organizationName; + $ret['description'] = $this->organizationName; + } + if (!empty($this->organizationDisplayName)) { + $ret['name'] = $this->organizationDisplayName; + } + if (!empty($this->tags)) { $ret['tags'] = $this->tags; @@ -654,8 +659,6 @@ class SimpleSAML_Metadata_SAMLParser { $ret = $this->getMetadataCommon(); - $ret['name'] = $this->entityId; - /* Find IdP information which supports the SAML 2.0 protocol. */ $idp = $this->getIdPDescriptors(self::$SAML20Protocols); @@ -692,11 +695,12 @@ class SimpleSAML_Metadata_SAMLParser { $slo = $this->getDefaultEndpoint($idp['singleLogoutServices'], array(self::SAML_20_REDIRECT_BINDING)); if($slo !== NULL) { $ret['SingleLogoutService'] = $slo['location']; - } - - /* If the response location is set, include it in the returned metadata. */ - if($slo['responseLocation']) { - $ret['SingleLogoutServiceResponse'] = $slo['responseLocation']; + + /* If the response location is set, include it in the returned metadata. */ + if(array_key_exists('responseLocation', $slo)) { + $ret['SingleLogoutServiceResponse'] = $slo['responseLocation']; + } + } diff --git a/modules/cron/www/cron.php b/modules/cron/www/cron.php index 6a2190cd85b6a0c5cea7f85aa2907bab61dc68c2..2f973fbe66c8f82bdf91051221caf89dbfc494e3 100644 --- a/modules/cron/www/cron.php +++ b/modules/cron/www/cron.php @@ -18,6 +18,8 @@ if (!is_null($cronconfig->getValue('allowed_tags'))) { } } + + $summary = array(); $croninfo = array( 'summary' => &$summary,