✨ 이메일 인증 로직#29
Conversation
ChanHoHan
requested review from
a team,
codero225,
dnjsals45,
jyh0521,
psychology50 and
rkskekzzz
There was a problem hiding this comment.
와, 이걸 이렇게 써버리네요 ㅋㅋㅋㅋㅋㅋ 재밌는 거 배워갑니다.
| "/api/v1/contents/get", "api/v1/comments/get", | ||
| "/api/v1/profile/**", | ||
| "/login/oauth2/**", "/api/v1/feed/title", "/api/v1/feed/body", "/api/v1/feed/nickname" | ||
| "/login/oauth2/**", "/api/v1/feed/title", "/api/v1/feed/body", |
There was a problem hiding this comment.
이게 뭐하는 api인지는 모르겠지만, /api/vi/feed/title, /api/v1/feed/body보다 QueryParam으로 받는 게 낫지 않을까요?
예를 들어, /api/v1/feed?type=title이라던가. 그러면 api도 하나로 줄일 수 있을 것 같아서요.
There was a problem hiding this comment.
근데 상수면 ENUM 쓰는 게 좋을 것 같은데, 전부 class를 사용하신 이유가 있는 건지 궁금해요
| OrganizationService organizationService) { | ||
| this.listOperations = redisTemplate.opsForList(); | ||
| this.redisTemplate = redisTemplate; | ||
| this.organizationService = organizationService; |
There was a problem hiding this comment.
OrganizationService가 Redis Service를 참조하는 게 아니라,
Redis Service가 OrganizationService를 컴포지션하고 있네요..?
나중에 의존 관계가 헷갈리지 않을까..살짝 우려되는 부분
| private static final long EXPIRED_TIME = 600; | ||
|
|
||
| public OrganizationCertificationService(RedisTemplate<String, OrganizationCertification> redisTemplate, | ||
| OrganizationService organizationService) { |
There was a problem hiding this comment.
이건 궁금한 건데, @Autowired 안 붙여줘도 자동으로 Bean 주입이 되나요???
| * @param httpServletResponse | ||
| * @throws Exception | ||
| */ | ||
| @GetMapping("/cert/{userId}/{token}/{code}") |
There was a problem hiding this comment.
이것도 path param이 아니라 query param이어야 할 거 같아요.
REST에서 제시한 URL과 좀 어긋나기도 하고 , 클라이언트가 /cert/1/sdfasdf/9515 이런 식으로 전달하려면 힘들 것 같습니다.
그리고 id를 받으려면 /users/{userId}, /users/{userId}/certs/{certId} 이런 식이어야 할 거예요.
2 participants
작업 이유
조직 이메일 인증 기능
작업 사항
조직 이메일 인증
조직 정보
추가로 해야할 일
이슈 연결
close #26