Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: hg security 의존성 제거 및 db 연결 테스트 컨트롤러 작성 #48

Merged
merged 3 commits into from
Dec 10, 2023

Conversation

dev-khg
Copy link
Contributor

@dev-khg dev-khg commented Dec 10, 2023

현재, batch에서 따로 security를 사용하는 부분이 없어 security 의존성을 제거하였습니다.
추가로, DB 연결 테스트를 위한 "/db/test" path가 추가되었으며 추후 삭제 될 예정입니다.

build.gradle Show resolved Hide resolved
@dev-khg dev-khg merged commit 40cdbeb into dev Dec 10, 2023
2 checks passed
spring.datasource.driver-class-name=org.mariadb.jdbc.Driver

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

주어진 코드 패치에 대해 간단한 코드 리뷰를 도와드리겠습니다.

주어진 코드 패치는 오류나 버그의 위험은 보이지 않습니다. 변경된 내용으로 보면 spring.datasource.driver-class-name 속성의 값이 동일한 것으로 보입니다. 라인의 끝에 개행 문자가 포함되지 않아도 두 줄 모두 동작할 것입니다.

코드 리뷰에서 개선사항은 발견되지 않습니다. 적용된 패치는 올바른 드라이버 클래스를 사용하는 것으로 보입니다.

요약하면, 주어진 코드 패치는 버그나 위험한 부분이 없으며 개선 사항도 없습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants