[개발일기] 로그인 리팩토링 (조언 환영)

[@toronrun]

우연히 들러보았는데 공부 기록들이 멋지네요. handler들이 모두 get으로 되어 있는데 보안상 좋지 않습니다. login, logout 모두 post로 변경하셔야 하고 이유는 찾으면 나오니 한번 찾아보세요. go는 권장되는 style guide가 확실한 편이라 폴더나 파일도 go style guide에 맞춰서 바꾸면 좋을 것 같네요.

[@rainbowbear0_0]

감사합니다! 로그인, 로그아웃은 post가 보안상 좋다는 것을 어디서 한 번 본 것 같은데 get을 사용하였네요! 자세한 이유는 한 번 더 찾아서 확실하게 기억하고 적용하도록 하겠습니다!
go에 권장되는 style guide가 있었군요! 오늘 바로 확인하고 수정하도록 하겠습니다! 좋은 정보 감사합니다!

[@toronrun]

작게 시작해서 리팩토링하는 모습도 좋아보입니다. 공부했던 것 바탕으로 적용하면서 스스로 필요성을 느끼고 수정하면 더 많이 배우니까요. 다만 이미 아키텍쳐에 대해서도 공부하시는것 같으니까 클린 아키텍쳐는 아니더라도 레이어드 아키텍쳐를 참조하시면 좀 더 구조화된 프로젝트가 될 거 같아요. 다른 코드 참조없이 혼자서 다 하겠다고 마음 먹어도 폴더 구조만이라도 참조하시면 좋을 것 같네요. 완성될때까지 화이팅입니다.

[@rainbowbear0_0]

레이어드 아키텍처 패턴은 전혀 몰랐습니다! 좋은 조언 해주셔서 감사합니다!
레이어드 아키택처 패턴도 확인해보고 적용하도록 노력하겠습니다!

[@fovidhakkxj]

보안은 get,post 와 같은 http method 와는 관련이 전혀 없습니다. 네트워크 레이어에서 종단간 패킷을 암호화를 하냐 안하냐가 중요합니다
http method 는 어떤 행위인가에 더 초점이 맞춰져있습니다.
로그인 로그아웃 등은 사용자의 요청을 통해 특정 action이 일어나길 바라는것이기에 post 가 적절합니다.
get 은 method 명 그대로 특정 리소스를 가져오는데 적합하겠죠??
코딩 공부 화이팅 입니다

[@rainbowbear0_0]

post가 get보다 보안적으로 좋다는 글을 봤는데 잘못된 정보였나봅니다!
로그인은 action이기에 post가 사용된다는 점 조언 감사합니다!
관련 자료를 조사해서 기록하겠습니다!

[@beungjinrho1292]

Redirect에 localhost로 하드코딩되어 있네요. Env파일이이나 다른 방식을 도입해서 서비스별로 적용되게 구성하는게 좋을거 같습니다.

[@rainbowbear0_0]

감사합니다! Env를 활용하면 나중에 배포를 하게 되었을 때 도메인을 구매하면 변경이 편리하겠네요! 바로 수정하도록 하겠습니다!
혹시 제가 생각한 이유 이외에 다른 이유가 있을까요?

[@티라노-y5h]

왜 go로 하시나요?

[@rainbowbear0_0]

블록체인에 관심이 있어 golang을 입문하였다가 빠지게 되었습니다.ㅎㅎ