Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
ultrakain opened this issue Aug 29, 2012 · 4 comments
Labels

Comments

@ultrakain
Copy link

계정 정보를 수정하는 화면 (사용자 설정 변경)에서 "현재 비밀번호", "새 비밀번호"를 넣지않고 "저장하기"를 누르면 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
Copy link
Member

doortts commented Aug 29, 2012

서버 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 (#15).

@eungjun-yi
Copy link
Member

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

@ultrakain
Copy link
Author

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

@eungjun-yi
Copy link
Member

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants