Fix/#320 네트워크 통신 로직수정 및 의존성 수정

[gnksbm]

작업내용

NetworkService

• URLSessionTask 리턴 Observable dispose 시점에 취소
• 에러 핸들링 추가

의존성 수정

• Firebase
  • App 모듈의 Firebase 패키지 제거
  • Data 모듈 Firebase 패키지 추가
• GoogleUtilities
  • Module name "GoogleUtilities-Privacy" is not a valid identifier 에러로 빌드가 되지 않는 문제 발생
  • 동일한 증상의 Issue를 리서치 하였고 7.13.1 버전에서 문제가 잘 동작함을 확인
    • issue with 7.13 version google/GoogleUtilities#146
  • 버스어디는 Firebase 10.23.1 버전을 사용하고 있고 Firebase 10.23.1는 GoogleUtilities를 7.12.1 ..< 8.0.0 버전으로 사용중
  • GoogleUtilities를 7.31.1 버전 고정으로 의존성 추가

리뷰요청

• @MUKER-WON
  • Firebase SPM 패키지 추가가 App 모듈에서 되고있어 Data 모듈로 수정하였는데 App 모듈에서 추가하신 이유가 있으신가요?

관련 이슈

close #320

[MUKER-WON]

@gnksbm 잘 기억은 안나는데 애널리틱스를 추가할 때 추가한거라면.. 앱 전체에 적용하는거니까 앱 모듈에 추가했나 보네요.. 외부 서비스는 데이터 레이어에 추가하면 되는걸까요?

[gnksbm]

@MUKER-WON App 모듈이 Data 모듈을 의존하다보니 Data의 패키지 의존성에 Firebase가 추가된다면 App 모듈도 패키지를 의존할 수 있지 않을 까 하는 생각이었어요
의존성 수정하고 빌드에 문제가 없어서 여쭤보았습니다!

[MUKER-WON]

@gnksbm 넵 크게 의존하는거 보단 작게 의존하는게 더 좋은거 같아요. 제가 아직 코드를 못봐서 빠르게 확인하고 코멘트 남기겠습니다.

[MUKER-WON]

@gnksbm
이제야 확인하네요 늦어서 죄송합니다.. 요즘 너무 정신없네요
GoogleUtilities 7.13.1 고정 하신이유 찾아봤고 확인했습니다.
NetworkService 부분 로직 에러처리 하신거랑 모듈 의존성 나눈거 확인했고 코드와 빌드에 문제 없어 머지하겠습니다.