nforge / devnote

developer's note for distributed env
69 stars 27 forks source link

계정 정보 수정화면에서 "현재 비밀번호", "새 비밀번호"를 넣지않고 수정시 500 에러 발생 #15

Open ultrakain opened 12 years ago

ultrakain commented 12 years ago

계정 정보를 수정하는 화면 (사용자 설정 변경)에서 "현재 비밀번호", "새 비밀번호"를 넣지않고 "저장하기"를 누르면 500 에러 발생하는 증상입니다.

에러 내용은 아래와 같습니다.

500 Error: Too short password! at _encodePassword (/Users/ultrakain/Documents/workspace_node/devnote/lib/users.js:68:13) at Object.changePassword (/Users/ultrakain/Documents/workspace_node/devnote/lib/users.js:79:31) at exports.postId (/Users/ultrakain/Documents/workspace_node/devnote/userApp.coffee:101:20) at callbacks (/Users/ultrakain/Documents/workspace_node/devnote/node_modules/express/lib/router/index.js:171:11) at param (/Users/ultrakain/Documents/workspace_node/devnote/node_modules/express/lib/router/index.js:145:11) at param (/Users/ultrakain/Documents/workspace_node/devnote/node_modules/express/lib/router/index.js:142:11) at pass (/Users/ultrakain/Documents/workspace_node/devnote/node_modules/express/lib/router/index.js:152:5) at Router._dispatch (/Users/ultrakain/Documents/workspace_node/devnote/node_modules/express/lib/router/index.js:179:5) at Object.router (/Users/ultrakain/Documents/workspace_node/devnote/node_modules/express/lib/router/index.js:39:10) at next (/Users/ultrakain/Documents/workspace_node/devnote/node_modules/express/node_modules/connect/lib/proto.js:190:15)

doortts commented 12 years ago

서버 validation에서 걸러내는 화면입니다.

그 이전에, 자바스크립트로 막아 놓았어야 하는데 막아 놓지 않아서 보게 되는 화면입니다.

view에서 validation하는 js 코드 추가해 놓겠습니다.

SW.CHAE

On Wednesday, August 29, 2012 at 3:02 PM, Lee Kwang Woon wrote:

계정 정보를 수정하는 화면 (사용자 설정 변경)에서 "현재 비밀번호", "새 비밀번호"를 넣지 않으면 500 에러 발생하는 증상입니다. 에러 내용은 아래와 같습니다. 500 Error: Too short password! at _encodePassword (/Users/ultrakain/Documents/workspace_node/devnote/lib/users.js:68:13) at Object.changePassword (/Users/ultrakain/Documents/workspace_node/devnote/lib/users.js:79:31) at exports.postId (/Users/ultrakain/Documents/workspace_node/devnote/userApp.coffee:101:20) at callbacks (/Users/ultrakain/Documents/workspace_node/devnote/node_modules/express/lib/router/index.js:171:11) at param (/Users/ultrakain/Documents/workspace_node/devnote/node_modules/express/lib/router/index.js:145:11) at param (/Users/ultrakain/Documents/workspace_node/devnote/node_modules/express/lib/router/index.js:142:11) at pass (/Users/ultrakain/Documents/workspace_node/devnote/node_modules/express/lib/router/index.js:152:5) at Router._dispatch (/Users/ultrakain/Documents/workspace_node/devnote/node_modules/express/lib/router/index.js:179:5) at Object.router (/Users/ultrakain/Documents/workspace_node/devnote/node_modules/express/lib/router/index.js:39:10) at next (/Users/ultrakain/Documents/workspace_node/devnote/node_modules/express/node_modules/connect/lib/proto.js:190:15)

— Reply to this email directly or view it on GitHub (https://github.com/nforge/devnote/issues/15).

eungjun-yi commented 12 years ago

근데 비밀번호를 수정하는 경우가 아니면, 비밀번호를 입력하지 않아도 되는게 맞을 것 같네요.

ultrakain commented 12 years ago

이 증상은 신규 사용자 등록 화면에서도 같습니다. ( 4자리 미만으로 입력 후 저장시 ) 1차로 클라이언트 자바스크립트에서 막고 2차로 서버쪽 자바스크립트에서도 체크해서 부드럽게 오류화면을 보여주면 좋을 것 같습니다.

eungjun-yi commented 12 years ago

개발모드에서만 call trace를 보여주고, production 모드에선 예쁜 에러페이지를 보여주는 게 어떨까 합니다.