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.

Sign up for GitHub

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

Jump to bottom

Fix/#182: 공지사항 크롤링 비동기처리 추가 및 무한 슬랙알림 문제 해결 #184

Merged
merged 4 commits into from
Feb 6, 2024
Merged

Fix/#182: 공지사항 크롤링 비동기처리 추가 및 무한 슬랙알림 문제 해결 #184

merged 4 commits into from
Feb 6, 2024

Commits on Feb 4, 2024

  1. fix(noticeHandler): 학과 공지사항 크롤링을 비동기로 처리하도록 변경 

    비동기처리 이외에도 기존에 함수단위의 트랜잭션을 학과 단위의 트랜잭션으로 축소
    @pp449
    pp449 committed Feb 4, 2024
    Configuration menu
    Copy the full SHA
    0941ac8 View commit details
    Browse the repository at this point in the history

  2. fix(cronNoticeCrawling): 크론 동작 시 에러 발생하면 3번까지 동작한 후 종료되도록 설정 

    만약 도중에 탈출하지 않으면 무한 루프로 인해 서버에 안좋은 영향 또는 무한 슬랙 알림을 받기에 3회까지 시도 후 종료되도록 설정
추가적으로 크론 내부에서 트랜잭션을 생성하지 않도록 변경
    @pp449
    pp449 committed Feb 4, 2024
    Configuration menu
    Copy the full SHA
    a4685c2 View commit details
    Browse the repository at this point in the history

  3. feat(noticeHandler): 예기치 못한 에러로 공지시항 크롤링 실패한 학과는 슬랙 알림을 받도록 추가

    @pp449
    pp449 committed Feb 4, 2024
    Configuration menu
    Copy the full SHA
    2daf2b6 View commit details
    Browse the repository at this point in the history

Commits on Feb 5, 2024

  1. fix(noticeHandler): 피드백 내용 반영 

    불필요한 콘솔 로그 제거
얼리 리턴 위치 변경
필터 조건 간결화
    @pp449
    pp449 committed Feb 5, 2024
    Configuration menu
    Copy the full SHA
    708e1c6 View commit details
    Browse the repository at this point in the history