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

デバッグ用のログ送信で新旧すべてのDailySummary, ExposureWindowが送信されてしまう #657

Closed
keiji opened this issue Jan 7, 2022 · 0 comments
Assignees
Labels
enhancement 新しい機能や改善のリクエスト
Projects

Comments

@keiji
Copy link
Collaborator

keiji commented Jan 7, 2022

その機能リクエストは何らかの問題に関連しますか / Is your feature request related to a problem?

Legacy-V1では、ENから取得するExposureInformationは「接触通知が発生する原因になったもの」だったので、追記するだけで問題なかったが、ExposureWindow modeで得られるDailySummaryとExposureWindowは「これまで発生したすべて」となり、仮に2回接触が発生した場合、getExposureWindow()で戻るデータの中には1回目のものも含まれる。

デバッグ用のログ送信ではENから取得した情報すべてを送ってしまっているので、複数回の接触が来たときに正しいデータが送られない可能性がある。

解決策についてお書きください / Describe the solution you'd like

すでにUserRepositoryのSetExposureDataでは、現在保管されていないデータのみフィルタリングして追記するように変更してあるので、これに倣って、デバッグ用のログ送信でも新しいデータのみフィルタリングして送信するようにする。

あなたが考える代替案についてご説明ください / Describe alternatives you've considered

デバッグ用の機能なので対応しない。
現状のテスト過程に「接触履歴を削除」を実行する工程が含まれているので、異なる接触が混入することは少ないと考える。

その他 / Additional context

@keiji keiji added enhancement 新しい機能や改善のリクエスト COCOA2 labels Jan 7, 2022
@keiji keiji self-assigned this Jan 7, 2022
@keiji keiji mentioned this issue Jan 11, 2022
@keiji keiji closed this as completed Jan 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement 新しい機能や改善のリクエスト
Projects
No open projects
COCOA2
  
Awaiting triage
Development

No branches or pull requests

1 participant