Reject invisible display names

clbn · Feb 8, 2022 · 149269d · 149269d
1 parent a605f05
commit 149269d
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/src/redux/reducers/page-view.js b/src/redux/reducers/page-view.js
@@ -2,7 +2,7 @@ import _ from 'lodash';
 
 import * as ActionTypes from '../action-types';
 import * as ActionHelpers from '../action-helpers';
-import { getSummaryPeriod } from '../../utils';
+import { getSummaryPeriod, userParser } from '../../utils';
 
 const { request, response } = ActionHelpers;
 
@@ -12,7 +12,7 @@ const initialState = {
 
 export default function pageView(state = initialState, action) {
   const getPageByOffset = (offset) => (offset ? Math.floor(offset / 30 + 1) : 1);
-  const getUserOnResponse = () => _.find(action.payload.users, { username: action.request.username });
+  const getUserOnResponse = () => userParser(_.find(action.payload.users, { username: action.request.username }));
   const getFormattedName = (user) => user.screenName + (user.username !== user.screenName ? ' (' + user.username + ')' : '');
 
   switch (action.type) {

diff --git a/src/utils/index.js b/src/utils/index.js
@@ -108,9 +108,14 @@ export function getFullDate(timestamp) {
 export function userParser(user) {
   const newUser = { ...user };
 
+  // Remove invisible characters from display name
+  if (newUser.screenName) {
+    newUser.screenName = newUser.screenName.replace(/[\u180E\u2000-\u200F\u2060\u2800\u3000\u3164\u{1D159}]/gu, '');
+  }
+
   // Missing display name
-  if (!user.screenName) {
-    newUser.screenName = user.username;
+  if (!newUser.screenName) {
+    newUser.screenName = newUser.username;
   }
 
   return newUser;