-
Notifications
You must be signed in to change notification settings - Fork 6
Crashes when invalid character, such as |, is in the user's Discord username #5
Description
11/1 18:43:41 [11026] - error: [sso-discord] could not create user undefined: [[error:invalid-username, Mercurial | corpnewt.com]]
11/1 18:43:41 [11026] - error: /auth/discord/callback
Error: [[error:invalid-username, Mercurial | corpnewt.com]]
at userNameValid (/Users/dhinak/NodeBB/src/user/create.js:162:78)
at /Users/dhinak/NodeBB/node_modules/async/lib/async.js:718:13
at async.forEachOf.async.eachOf (/Users/dhinak/NodeBB/node_modules/async/lib/async.js:233:13)
at _parallel (/Users/dhinak/NodeBB/node_modules/async/lib/async.js:717:9)
at Object.async.parallel (/Users/dhinak/NodeBB/node_modules/async/lib/async.js:731:9)
at Object.User.isDataValid (/Users/dhinak/NodeBB/src/user/create.js:153:9)
at Object.User.create (/Users/dhinak/NodeBB/src/user/create.js:20:8)
at /Users/dhinak/Downloads/nodebb-plugin-sso-discord-alt-master/library.js:210:12
at Command.callback (/Users/dhinak/NodeBB/src/database/redis/sorted.js:198:4)
at normal_reply (/Users/dhinak/NodeBB/node_modules/redis/index.js:714:21)
at RedisClient.return_reply (/Users/dhinak/NodeBB/node_modules/redis/index.js:816:9)
at JavascriptRedisParser.returnReply (/Users/dhinak/NodeBB/node_modules/redis/index.js:188:18)
at JavascriptRedisParser.execute (/Users/dhinak/NodeBB/node_modules/redis-parser/lib/parser.js:491:12)
at Socket. (/Users/dhinak/NodeBB/node_modules/redis/index.js:267:27)
at emitOne (events.js:96:13)
at Socket.emit (events.js:188:7)
11/1 18:44:48 [11026] - error: [sso-discord] could not create user undefined: [[error:invalid-username, Mercurial | corpnewt.com]]
11/1 18:44:48 [11026] - error: /auth/discord/callback
Error: [[error:invalid-username, Mercurial | corpnewt.com]]
at userNameValid (/Users/dhinak/NodeBB/src/user/create.js:162:78)
at /Users/dhinak/NodeBB/node_modules/async/lib/async.js:718:13
at async.forEachOf.async.eachOf (/Users/dhinak/NodeBB/node_modules/async/lib/async.js:233:13)
at _parallel (/Users/dhinak/NodeBB/node_modules/async/lib/async.js:717:9)
at Object.async.parallel (/Users/dhinak/NodeBB/node_modules/async/lib/async.js:731:9)
at Object.User.isDataValid (/Users/dhinak/NodeBB/src/user/create.js:153:9)
at Object.User.create (/Users/dhinak/NodeBB/src/user/create.js:20:8)
at /Users/dhinak/Downloads/nodebb-plugin-sso-discord-alt-master/library.js:210:12
at Command.callback (/Users/dhinak/NodeBB/src/database/redis/sorted.js:198:4)
at normal_reply (/Users/dhinak/NodeBB/node_modules/redis/index.js:714:21)
at RedisClient.return_reply (/Users/dhinak/NodeBB/node_modules/redis/index.js:816:9)
at JavascriptRedisParser.returnReply (/Users/dhinak/NodeBB/node_modules/redis/index.js:188:18)
at JavascriptRedisParser.execute (/Users/dhinak/NodeBB/node_modules/redis-parser/lib/parser.js:491:12)
at Socket. (/Users/dhinak/NodeBB/node_modules/redis/index.js:267:27)
at emitOne (events.js:96:13)
at Socket.emit (events.js:188:7)
node_redis: Deprecated: The HDEL command contains a "null" argument.
This is converted to a "null" string now and will return an error from v.3.0 on.
Please handle this in your code to make sure everything works as you intended it to.