riyenas0925 / Sejong_Track_Management

:school: 세종대학교 트랙관리 프로그램
Apache License 2.0
6 stars 2 forks source link

ENH : 데이터베이스 H2로 변경, application.properties 선택할수 있게 변경 #164

Closed riyenas0925 closed 4 years ago

riyenas0925 commented 4 years ago
riyenas0925 commented 4 years ago

@kimhanui 이거 어프로브 머지 해보고 아래 주소 함 접속해서 정상작동하는지 확인 부탁드림다 https://tracksejong.herokuapp.com/

만약 된다면 이제 pr이 머지 될때(develop에 새로운 커밋이 들어올때) 아마 아래 주소로 배포가 자동으로 되서 확인해볼수 있을듯?

riyenas0925 commented 4 years ago

@kimhanui 배포 하다가 appCrash라길래 확인해보니까 아래 오류라는데 확인 부탁~

2020-03-12T14:48:12.574517+00:00 app[web.1]: ***************************
2020-03-12T14:48:12.574518+00:00 app[web.1]: APPLICATION FAILED TO START
2020-03-12T14:48:12.574518+00:00 app[web.1]: ***************************
2020-03-12T14:48:12.574518+00:00 app[web.1]:
2020-03-12T14:48:12.574519+00:00 app[web.1]: Description:
2020-03-12T14:48:12.574519+00:00 app[web.1]:
2020-03-12T14:48:12.574520+00:00 app[web.1]: The dependencies of some of the beans in the application context form a cycle:
2020-03-12T14:48:12.574520+00:00 app[web.1]:
2020-03-12T14:48:12.574521+00:00 app[web.1]: memberRestController defined in URL [jar:file:/app/target/sejongtrack-0.0.1-SNAPSHOT.jar!/BOOT-INF/classes!/kr/ac/sejong/web/MemberRestController.class]
2020-03-12T14:48:12.574521+00:00 app[web.1]: ┌─────┐
2020-03-12T14:48:12.574522+00:00 app[web.1]: |  customUserDetailsService defined in URL [jar:file:/app/target/sejongtrack-0.0.1-SNAPSHOT.jar!/BOOT-INF/classes!/kr/ac/sejong/service/CustomUserDetailsService.class]
2020-03-12T14:48:12.574523+00:00 app[web.1]: ↑     ↓
2020-03-12T14:48:12.574524+00:00 app[web.1]: |  springSecurityConfig (field kr.ac.sejong.service.CustomAuthenticationProvider kr.ac.sejong.config.SpringSecurityConfig.authProvider)
2020-03-12T14:48:12.574524+00:00 app[web.1]: ↑     ↓
2020-03-12T14:48:12.574524+00:00 app[web.1]: |  authProvider (field kr.ac.sejong.service.CustomUserDetailsService kr.ac.sejong.service.CustomAuthenticationProvider.customUserDetailsService)
2020-03-12T14:48:12.574525+00:00 app[web.1]: └─────┘
2020-03-12T14:48:12.574525+00:00 app[web.1]:
2020-03-12T14:48:12.574525+00:00 app[web.1]:
2020-03-12T14:48:12.727243+00:00 heroku[web.1]: State changed from starting to crashed
2020-03-12T14:48:12.710043+00:00 heroku[web.1]: Process exited with status 1
kimhanui commented 4 years ago

@riyenas0925 안그래도 그거 오류 확인하는 중 ㅠ 생성자 주입 쓰니까 빈 순환참조되는거 잡는듯 -> 어플리케이션 구동 아예 막아버림

https://yaboong.github.io/spring/2019/08/29/why-field-injection-is-bad/

해결방안은 세터 주입/필드 주입이 가장 쉽긴한데 임시방편이고 로직 재설계가 답이라 하네

riyenas0925 commented 4 years ago

@kimhanui 음... 지금 보면 회원가입 부분이 우리가 정한 규칙들(dto사용, private final로 주입하기 등등..)을 안지키고 있는게 있으니까 리펙터링 해서 변경하자 전에 말한 기본권한 학생으로 부여하는것도 해서 ㅇㅇ

riyenas0925 commented 4 years ago

@kimhanui 근데 내 로컬에서 돌릴때는 문제 없었는데 서버에 올리니까 에러가 발생했네?? 너도 로컬 환경에서는 오류 난건가?

kimhanui commented 4 years ago

@riyenas0925 로컬에서 문제 없엇음? 왜지 난 로컬ㄹ에서 에러 남 ㅇㅇ

riyenas0925 commented 4 years ago

@kimhanui 아그래?? 내꺼에서는 오류가 없었는데 뭐지 ㅋㅋ 아 글고 일단 이 pr이 H2데이터베이스로 변경한거니까 확인해보고 어프로브 부탁드림다

kimhanui commented 4 years ago

@riyenas0925 이거 해결하려다가 나머지거 다 못할거 같은데 일단 @Inject주입으로 바꿔놓고 다음주에 로직 재설계 하는거 어떰?