Merge pull request #1830 from thecashewtrader/register-no-email-note

Add a note if user attempts to register with (apparently) an email

src/components/RegisterPage.vue

@@ -33,6 +33,8 @@
 </template>
 
 <script>
+import { isEmail } from "../utils/Misc.js";
+
 export default {
     data() {
         return {
@@ -52,6 +54,8 @@ export default {
     methods: {
         register() {
             if (!this.username || !this.password) return;
+            if (isEmail(this.username) && !confirm(this.$t("info.register_no_email_note"))) return;
+
             this.fetchJson(this.authApiUrl() + "/register", null, {
                 method: "POST",
                 body: JSON.stringify({

src/locales/en.json

@@ -173,6 +173,7 @@
         "page_not_found": "Page not found",
         "copied": "Copied!",
         "cannot_copy": "Can't copy!",
-        "local_storage": "This action requires localStorage, are cookies enabled?"
+        "local_storage": "This action requires localStorage, are cookies enabled?",
+        "register_no_email_note": "Using an e-mail as username is not recommended. Proceed anyways?"
     }
 }

src/utils/Misc.js

@@ -0,0 +1,8 @@
+export const isEmail = input => {
+    // Taken from https://emailregex.com
+    const result = input.match(
+        //eslint-disable-next-line
+        /^(([^<>()\[\]\\.,;:\s@"]+(\.[^<>()\[\]\\.,;:\s@"]+)*)|(".+"))@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}])|(([a-zA-Z\-0-9]+\.)+[a-zA-Z]{2,}))$/,
+    );
+    return result;
+};