Remove any mailto: prefixes when set in technicalcontact email config.

People might do this in the past to get correct metadata (thereby
breaking other email related functions). But our metadata generator
will now take care of this, so we can just ignore any mailto: prefix.

lib/SimpleSAML/Utils/EMail.php

@@ -66,6 +66,7 @@ class EMail
     {
         $config = Configuration::getInstance();
         $address = $config->getString('technicalcontact_email', 'na@example.org');
+        $address = preg_replace('/^mailto:/i', '', $address);
         if ('na@example.org' === $address) {
             throw new \Exception('technicalcontact_email must be changed from the default value');
         }

tests/lib/SimpleSAML/Utils/EMailTestCase.php

@@ -130,4 +130,26 @@ class EMailTestCase extends ClearStateTestCase
         $this->expectException(\InvalidArgumentException::class);
         $email->setTransportMethod('smtp');
     }
+
+    /**
+     * Test setting configuration.
+     *
+     * @return void
+     */
+    public function testGetDefaultMailAddress()
+    {
+        Configuration::loadFromArray([
+            'technicalcontact_email' => 'gamaarna@example.org',
+        ], '[ARRAY]', 'simplesaml');
+
+        $mail = new EMail('test', null, 'phpunit@simplesamlphp.org');
+        $this->assertEquals('gamaarna@example.org', $mail->getDefaultMailAddress());
+
+        Configuration::loadFromArray([
+            'technicalcontact_email' => 'mailto:gamaarna@example.org',
+        ], '[ARRAY]', 'simplesaml');
+
+        $mail = new EMail('test', null, 'phpunit@simplesamlphp.org');
+        $this->assertEquals('gamaarna@example.org', $mail->getDefaultMailAddress());
+    }
 }