fix: Validates emails before using them to identify users. (#229)

xAPI-vle · Aug 28, 2018 · 7dca1ae · 7dca1ae
1 parent 2da2e11
commit 7dca1ae
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/transformer/utils/get_user.php b/src/transformer/utils/get_user.php
@@ -19,8 +19,9 @@
 
 function get_user(array $config, \stdClass $user) {
     $fullname = get_full_name($user);
+    $hasvalidemail = filter_var($user->email, FILTER_VALIDATE_EMAIL);
 
-    if (array_key_exists('send_mbox', $config) && $config['send_mbox'] == true) {
+    if (array_key_exists('send_mbox', $config) && $config['send_mbox'] == true && $hasvalidemail) {
         return [
             'name' => $fullname,
             'mbox' => 'mailto:' . $user->email,