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

接触通知の発生イベントを収集する処理の実装 #1038

Merged
5 commits merged into from
Jun 20, 2022

Conversation

ghost
Copy link

@ghost ghost commented Jun 15, 2022

Issue 番号 / Issue ID

  • Close #

目的 / Purpose

  • 接触通知の発生イベントを収集する処理を実装する
    • 当PRでは画面の修正は含まない
    • APIの修正は含まない

変更内容 / Changes

  • 接触通知の発生イベントを収集する処理を実装
  • 実装方法は、FYI: 接触通知の発生イベントを収集する準備 #1023 を踏襲。主に以下の内容を追加対応。
    • リトライ間隔を指定(3回リトライで、1秒→2秒→4秒間隔に調整)
    • エラーハンドリングを修正
    • 通知時に非許可だったログが送信時に許可されていると送信対象になっていたので修正
  • デバイス認証時にPayloadがエラーの際もリトライを実施するようにした
    • 処理番号登録時と同様の仕様(3回リトライで、4秒→8秒→16秒間隔)

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

[ ] Yes
[x] 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

  • [ ]

その他 / Other information


Internal IDs:

  • Product Backlog Item 6037

@ghost ghost requested review from a user, cocoa-dev003 and cocoa-dev009 June 15, 2022 06:51
@ghost ghost self-assigned this Jun 15, 2022
@ghost
Copy link
Author

ghost commented Jun 16, 2022

  • リトライの処理を調整
  • デバイス認証時のPayloadエラー時のリトライを追加(処理番号登録時の仕様に合わせた)
  • リトライの処理はPolly(NuGetパッケージ)を利用するようにした

@sonarcloud
Copy link

sonarcloud bot commented Jun 16, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

47.0% 47.0% Coverage
0.0% 0.0% Duplication

@ghost ghost force-pushed the feature/send-exposure-notify branch from 979b7a6 to 642f67e Compare June 17, 2022 02:03
@ghost
Copy link
Author

ghost commented Jun 20, 2022

iOSで24時間おきのバックグラウンド処理での送信が発生していない。(土日に放置していたが動作した痕跡がない)
原因もすぐにはわからなそうなので、一旦マージし別で対応します。

@ghost ghost merged commit 091b5d0 into develop Jun 20, 2022
@ghost ghost deleted the feature/send-exposure-notify branch June 20, 2022 02:51
This pull request was closed.
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.

1 participant