newuser.js logic issue
From
Nightfox@VERT/DIGDIST to
Digital Man on Sat Oct 20 10:41:59 2012
Hi DM -ããWhile testing my new user script, I think I may have found a logic issue thatãexists in the stock newuser.js. In the function chk_qwk_id(), the last checkãis as follows:ãif(system.matchuser(str,true)!=user.number || system.trashcan(str))ããI think the purpose of the "system.matchuser(str,true)!=user.number" is to makeãsure that the username entered belongs to the current user. However, if theãnewuser.js script needs to ask for a valid QWK ID (i.e., because theãusername/alias entered by the user is not a valid QWK ID), thenãsystem.matchuser() will probably return 0 because it's not found. Since 0 isãdifferent from user.number, anything that the user enters for a QWK ID upon newãuser registration will be rejected.ããNightfoxããã---ã þ Synchronet þ Digital Distortion BBS - digitaldistortionbbs.comã
From
Digital Man@VERT to
Nightfox on Sat Oct 20 13:14:50 2012
Re: newuser.js logic issueã By: Nightfox to Digital Man on Sat Oct 20 2012 10:41 amãã > Hi DM -ã >ã > While testing my new user script, I think I may have found a logic issueã > that exists in the stock newuser.js. In the function chk_qwk_id(), theã > last check is as follows:ã > if(system.matchuser(str,true)!=user.number || system.trashcan(str))ã >ã > I think the purpose of the "system.matchuser(str,true)!=user.number" is toã > make sure that the username entered belongs to the current user. However,ã > if the newuser.js script needs to ask for a valid QWK ID (i.e., because theã > username/alias entered by the user is not a valid QWK ID), thenã > system.matchuser() will probably return 0 because it's not found. Since 0ã > is different from user.number, anything that the user enters for a QWK IDã > upon new user registration will be rejected.ããThanks, should be fixed now.ãã digital manããSynchronet "Real Fact" #32:ãThe Synchronet web user interface was contributed by Robert Couture, Runemaster.ãNorco, CA WX: 68.6øF, 77.0% humidity, 4 mph SE wind, 0.01 inches rain/24hrsã---ã þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.netã