peoplefund-tech / react-naver-login

A Naver Login Component for React
MIT License
20 stars 9 forks source link

'browser' in process ERROR #12

Open moonseoklee opened 4 years ago

moonseoklee commented 4 years ago

if (!('browser' in process)) 구문으로 인해 정상적으로 실행이 되지 않았습니다. 제 CRA 앱의 process를 찍어보니 {title: "browser", env: {…}, argv: Array(0), version: "", ...} 가 나왔고 browser 키가 없어 return 되어 프로세스가 종료됩니다. not 연산자 ! 을 제거하면 정상적으로 동작합니다만 제 앱의 brower 키가 없는 정확한 이유를 몰라 이슈 남깁니다.

moonseoklee commented 4 years ago

https://www.npmjs.com/package/react-login-by-naver 제가 사용하기 위해 해당 오류를 수정하여 배포해놓았습니다. 문제되면 삭제하겠습니다.

tmmoond8 commented 4 years ago

감사합니다. 획인 후 반영하도록 하겠습니다. 늦게 답장 드려서 죄송합니다. 참여해주셔서 감사합니다.

On Mon, Jun 22, 2020, 10:45 PM leemoonseok notifications@github.com wrote:

https://www.npmjs.com/package/react-login-by-naver 제가 사용하기 위해 해당 오류를 수정하여 배포해놓았습니다. 문제되면 삭제하겠습니다.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/peoplefund-tech/react-naver-login/issues/12#issuecomment-647529051, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACW7ZZG54GIIWRHFLNVOVCTRX5N6PANCNFSM4ODMSLUQ .

moonseoklee commented 4 years ago

추가로 해당 if 문을 if(typeof window===undefined) 로 바꿔 배포해놓았습니다. 아마 제 웹의 로그인 Component가 서버사이드에서 rendering 되어 에러가 났던 것으로 추측됩니다!

tmmoond8 commented 4 years ago

모듈에 반영하여 배포 하였습니다. 감사합니다.