Skip to content
This repository has been archived by the owner on Apr 12, 2023. It is now read-only.

HttpClientインスタンスの取り回しを効率化する #951

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

keiji
Copy link
Collaborator

@keiji keiji commented Mar 31, 2022

Issue 番号 / Issue ID

目的 / Purpose

サービス毎にHttpClientのインスタンスを持っていたが、HttpClientはコネクションプールを持つので効率的ではない。
アプリが使うHttpClientのインスタンスをHttpClientServiceに集約して無駄なHttpClientインスタンスを作成しないようにする。

変更内容 / Changes

  • HttpDataServiceを廃止してHttpClientServiceに集約する
  • HttpClientServiceは必要なHttpClientを各サービスに提供する役割
    • ApiClient: APIにアクセスするのに使う。FrontDoorを超えられる
    • StaticJsonContentClient: Jsonの静的コンテンツを取得するのに使う。タイムアウト10秒。FrontDoorは超えられない
    • CdnClient: CDNからデータをダウンロードするのに使う

破壊的変更をもたらしますか / Does this introduce a breaking change?

[x] Yes
[ ] No

Pull Request の種類 / Pull Request type

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

確認事項 / What to check

  • ApiClient/StaticJsonContentClientのヘッダー指定が正しいか再確認(FrontDoorの向こう側にあるものにStaticJsonContentClientを指定していないか

その他 / Other information


Internal IDs:

@keiji keiji self-assigned this Mar 31, 2022
@keiji keiji changed the title Refactor/httpclient HttpClientインスタンスの取り回しを効率化する Mar 31, 2022
@keiji keiji marked this pull request as ready for review March 31, 2022 07:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HttpClientの取り回しを再検討
1 participant