스프링 시큐리티 인증 서버 + 게시판 구현 잘된건지 검토좀 부탁 드려여 ! ^^;

Question

스프링 시큐리티로 인증 인가 &#43; 게시판 프로젝트를 만들었습니다. 질문: 잘구현된 되었는지 msa 의 인증 인가 &#43; 게시판 서비스로 사용해도 될것 같은지 무조건 고쳐야할 부분이나 구조적인 문제등을 한번 봐주세요 이후 msa 서비스의 일부로 사용 가능할지도 한번 검토해주시면 감사요 !!구현한것: User, Role 모델과 연동한 스프링 시큐리티 인증, 인가 시스템 상세: 스프링 시큐리티로 로그인, 로그인 체크, role 별 권한 체크 서버 시작하면 테스트 데이터 자동 입력 게시판 api mysql (docker) 컨트롤러: https://github.com/hyunsokstar/ohs-backend-test/blob/main/src/main/java/org/zerock/apiserver/controller/AuthController.java 서비스: https://github.com/hyunsokstar/ohs-backend-test/tree/main/src/main/java/org/zerock/apiserver/service 데이터 자동 입력: https://github.com/hyunsokstar/ohs-backend-test/tree/main/src/main/java/org/zerock/apiserver/initializer security 관련: https://github.com/hyunsokstar/ohs-backend-test/tree/main/src/main/java/org/zerock/apiserver/security dtos: https://github.com/hyunsokstar/ohs-backend-test/tree/main/src/main/java/org/zerock/apiserver/dto/auth 테스트: 1. 로그인 테스트: 로그인 확인: role 이 teacher 일 경우 ( /admin/xx 에 대해 접근 제한 , 토큰으로 유저 식별후 판단 )

82savage · Answer

우선 MSA 로 추후 확장성을 원하신다면 추후에는 인증 과 게시판은 분리 하시고 서로 통신하는 구조로 생각 하셔야 합니다.그리고 로그인 시, 왜 리퀘스트 객체와 와 테스트 데이터를 검증 하는 로직이 없나요!로그인에서 가장 중요한것은 토큰 발급이 아니라 객체의 검증 입니다!AuthService를 구현하셔서 리퀘스트된 계정과 패스워드를 입력 해놓은 테스트 데이터와 일치 하는지 검증을 하시고토큰을 발급 해주세요.그리고 발급된 리프레시 토큰은 생명주기를 넣어서 레디스에 한번 저장해보세요.사실 코드 문법 스타일 이나 계층 구조에 대해선 하고싶은 말이 있어도 쉽게 못하긴 해요 코드가 정답은 없으니까요.개인적인 생각으로는 더 개선 될수 있는 부분이 있는거 같습니다.AuthenticationException 로 핸들러에서 충분히 처리가 가능한 부분이고log를 찍고 API를 실행 해보는것도 좋긴 한데테스트 코드를 활용하시는게 훨씬 좋아 보입니다.그리고 학습을 하시다 보니 구글이나 다른 gitHub를 많이 참고 하셨을거 같은데그대로 사용하시기 보단 조금더 리팩토링을 해보는것도 개발 하면서 재밌으실 거에요간단한 예) 토큰을 추출 하는 로직 private String extractToken(HttpServletRequest request) { String bearerToken = request.getHeader("Authorization"); if (bearerToken != null && bearerToken.startsWith("Bearer ")) { return bearerToken.substring(7); } return null; }리팩 토링 후 - 하드코딩을 따로 관리하고, 함수를 빠져 나갈수 있는 예외처리를 먼저 함 private String extractToken(HttpServletRequest request) { String token = request.getHeader(CommonConstant.TOKEN_HEADER); if (StringUtils.isEmpty(token)) { return StringUtils.EMPTY; } return token.replaceAll(CommonConstant.TOKEN_TYPE,"").trim(); }취향 차이긴 합니다. ㅎㅎ코드를 계속 피드백 받으려 노력하시는 모습 보기 좋습니다 화이팅 하세요!

스프링 시큐리티 인증 서버 + 게시판 구현 잘된건지 검토좀 부탁 드려여 ! ^^;

1개의 답변

#login 목록