Dev to Main week3 #130
Merged
Dev to Main week3 #130
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* init: 의존 추가 - SnakeYAML 의존을 추가했습니다. - 기존 의존 버전을 최신화했습니다. * style: AppKeyTest 스타일 변경 - 테스트 스타일을 변경했습니다. * chore: 클래스 이름 변경 - LogbatOptions -> LogBatOptions 이름 변경 * refactor: LogBatOptions 생성 로직 변경 - 생성 로직을 Config을 전달받아 구현하는 방식으로 수정했습니다. - LogBatconfig -> LogBatConfig 클래스 명 변경을 진행했습니다. - 이에 따른 테스트를 수정했습니다. * refactor: LogBatConfig 로직 변경 - Map<String, String>을 관리하도록 수정했습니다. - 이에 따른 테스트를 수정했습니다. * chore: 클래스 명 변경 및 docs 추가 - InvalidAppKeyException -> InvalidOptionException - LogBatConfig docs 추 * docs: AppKey 주석 작성 - AppKey에 대한 주석을 작성했습니다. * refactor: LogBatFactory 로직 변경 - 싱글턴 보장을 위한 VarHandler 사용 로직을 적용했습니다. - javadoc을 추가했습니다. * chore: 클래스 이름 변경 - LogbatAppender -> LogBatAppender * refactor: LogSender 로직 변경 - 생성자에서 LogBatOptions으로만 생성이 가능하게 수정했습니다. - 내부 메서드에 대한 리팩토링을 진행했습니다. - LogSenderTest를 수정했습니다. * refactor: LogBatConfigLoader 로직 변경 - .yml, .yaml 파일 로드시 SnakeYAML 의존을 사용하도록 수정했습니다. - java-doc을 추가했습니다. * chore: 클래스 이름 변경 - logBat -> LogBat 클래스 이름을 변경했습니다. - Java-Doc에 저자를 추가했습니다. * docs: LogProcessScheduler - Java-Doc을 추가했습니다. * chore: 로직 수정 - null -> .loadConfig() * refactor: VarHandle 변수명 변경 - INSTANCES -> INSTANCE * refactor: instance volatile 적용 * refactor: `compareAndSet` 을 사용한 객체 할당 - `synchronized` 키워드 제거 - `compareAndSet` 를 통해서 하나의 객체만 반환될것을 보장 * feat: 다중 값 허용하지 않도록 변경 * refactor: filename 에 final 키워드 적용 * refactor: 매개변수의 filename 제거 - 매개변수의 filename 없이 객체 내부의 값을 참조하도록 변경 * fix: 여러 설정파일로 부터 값을 불러올 수 있도록 변경 * style: 코드 위치 변경 - VarHandle 부분을 코드 최하단으로 옮겼습니다. - docs를 수정했습니다. * chore: LogBatConfigLoader - 일부 로직 수정 - docs 수정 * style: LogBufferTest - 테스트 형식을 이전 테스트와 동일하게 맞췄습니다. --------- Co-authored-by: luizy <[email protected]>
* feat: 로그 요청 유효성 검사를 위한 커스텀 어노테이션 추가 - ValidLogRequests 어노테이션 생성 * feat: 로그 요청 목록 유효성 검사 기능 구현 - LogRequestsValidator 클래스 생성 - 빈 요청, 개별 요청, 전체 요청의 유효성 검사 로직 구현 - 유효하지 않은 요청 필터링 및 오류 메시지 생성 기능 추가 * test: 여러 로그 POST 요청에 대한 테스트 케이스 추가 - 빈 요청 시 400 에러 반환 테스트 - 모든 요청 무효 시 400 에러 반환 테스트 - 일부 요청만 유효한 경우 정상 처리 테스트 - 전체 요청 유효 시 정상 처리 테스트 * test: 빈 줄을 허용하는 비즈니스 로직 변화로 테스트 코드 수정 - data가 빈 줄일 때에 대한 예외상황을 허용하도록 변경 * feat: LogController에 로그 요청 유효성 검사 적용 - @ValidLogRequests 어노테이션을 LogController의 saveLogs 메서드에 적용 * refactor: indexOf 연산을 사용하지 않기 위한 리팩토링 - requests로 stream을 돌리지 않고 ,IntStream을 통해 index 기반 접근하여 반환하도록 수정했습니다.
도커라이징하지 않고, jar를 직접 배포하는 방식으로 변경합니다.
* feat: AppCache 삭제 API 구현 - AppCache 삭제를 진행하는 API를 구현했습니다. * feat: AppService .evictAppCache 메서드 구현 - 앱 캐시 삭제 메서드를 구현했습니다. - 앱이 존재하는 경우 예외를 발생시킵니다. * refactor: API 전달 형식 변경 - Body -> Path Parameter * refactor: 앱 삭제시 캐시 삭제 API 호출 로직 추가 - 앱 삭제 요청시 API 호출을 통한 캐시 삭제 로직을 추가했습니다. * remove: 사용하지 않는 테스트 삭제
* chore: 사용하지 않는 클래스 Deprecated 처리 - 최적화를 위해 사용하지 않는 클래스에 대해 Deprecated 처리를 진행했습니다. - 이에 따른 로직 수정을 진행했습니다. * refactor: 저장 로직 변경 - .stream()을 사용하지 않는 방식으로 리팩토링 * refactor: List를 생성로직 추가 - List를 생성해서 repository로 전달 - BlockingQueue이기 때문에 접근 횟수를 줄임 * refactor: Log data는 비어있을 수 있다 * refactor: Controller 빈 배열 검사 추가 * test: 미사용 테스트 제거 * style: LogController - 린트 수정 * fix: 형식에 맞는 값을 제거하고 배열 저장 --------- Co-authored-by: luizy <[email protected]>
tidavid1
approved these changes
Aug 27, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Dev to Main week3