イベントの入場条件に「COOCAのインストール後14日経過＋接触記録がないこと」が存在し、3 の使用日数のリセットは厳しいかもしれませんね。

https://twitter.com/ls_9mermaid/status/1354433072477786117

個人的には 1 か 2 が良いと思います。

2 に加えて、MainActivityを起動したタイミングで新しいバージョンのデータが存在していた場合、Application.Current.Propertiesからは使用開始日のデータに限定してMigrateする。のような手当をすると、実装負荷とユーザーへの影響のバランスが良いと考えます。

今の状態では null 安全ではありません。不具合を減らす為には null 許容型/非許容型を有効にすべきでしょう。プロジェクト全体の設定を変えるのが大変であれば、’#nullable’ ディレクティブを使う事もできます。

追試用に @keiji が作った検証コードの Xamarin.Forms 版を置いておきます。

AndroidBackTaskTest/RequiresDeviceIdleTest at main · moonmile/AndroidBackTaskTest

御意。現状の v1.2.2 で UserUuid は取り除かれているので、ここはクローズしていいと思います。

この問題の原因は以下のようになっていて

• v1.2.1 以前で利用していた Application.Current.Properties は Xamarin.Forms に依存している
• Android では WorkManager の Worker が起動するタイミングでは Xamarin.Forms が初期化されていない可能性があるので Properties は読み書きできない
  • MainActivity を起動後、OS側でプロセスが終了させられなかった場合のみ偶然動く

このため現状のままだと Migrate を Worker から実行すること自体が困難ということになりそうです。解決案としては、

1. Migrate の処理だけ Xamarin.Forms に依存せずに直接 Properties ファイルを読み込んで実行する
2. Worker から (ExposureNotificationHandler から) Migrate するのはあきらめて、MainActivity が実行されたタイミングで移行する
3. v1.2.1 以前からのアップデート後、Migrate 未実行で Worker が起動された場合は Migrate をあきらめる(使用日数も 0日になる)

という案がありそうです。1. は iOS との差分をどう処理するかの問題があり、2. は Worker で ExposureNotificationService が呼ばれて新しい PreferencesService / SecureStorageService にデータが保存されてしまうので、MainActivity 実行時には Properties とマージする処理が必要になります。

v1.2.1 のユーザを切り捨てる判断ができるのであれば、3. の選択肢が一番楽なのかもしれません。 ただし、Worker の Migrate を単に削除してしまうと、Worker で更新した Preferences/SecureStorage の情報が Migrate で上書きされる状況になるので、アップデート後に接触通知があったユーザは起動後情報が消えるなどの問題を起こす可能性もありそうです(未検証)。 回避するには、Migrate 実行済みのフラグを PreferencesService に保存するなどの対応追加が必要ではないかと思われます。

#34 に別途 Issue を立てさせていただきましたが、BeaconService.cs は現状プロジェクト構成に含まれていないファイルで使用されていないと思います

Describe the bug リポジトリにCOCOAの初回リリース以前に使われていたと思われる不要なファイルが含まれています

Expected behavior 純粋に不要なファイルのためユーザ影響などはありません(バイナリサイズにも当然影響ありません)が、 検索などの対象になってしまうので邪魔なものは取り除きたいです。

Additional context わかっているものとしては以下のものがありますが網羅的に調べてはいないので過不足があるかもしれません

EN 以前の実装でバックグラウンド実行に使われていたらしいもの

Covid19Radar.Android/BackgroundService/

その他 UnUsed ディレクトリ

Covid19Radar/Behaviors/UnUsed Covid19Radar/Converters/UnUsed Covid19Radar/Model/UnUsed Covid19Radar/Templates/UnUsed Covid19Radar/ViewModels/HomePage/UnUsed Covid19Radar/ViewModels/Tutorial/UnUsed Covid19Radar/ViewModels/UnUsed Covid19Radar/Views/HomePage/UnUsed Covid19Radar/Views/Tutorial/UnUsed Covid19Radar/Views/UnUsed Covid19Radar.Android/Services/UnUsed Covid19Radar.iOS/Services/UnUsed

マージしなくても構いませんが、下記の様に大幅に改変しました事を報告させていただきます。 https://github.com/Takym/Covid19Radar お忙しい中大変恐縮ですが評価の程宜しくお願いします。

@Takym さん、ご連絡ありがとうございます。ご紹介頂いたリポジトリは今後の参考にさせて頂きます。

今後もお気づきの点がありましたらぜひ Issue にてご指摘ください。

コードへのリンクがmasterを参照しているため、Issue 記述当時とズレが生じている様に思います。 Issue 作成時点の日付から v1.1.1 を参照していると仮定し、v1.2.2 との差分を提示します。

• UserDataModel

  • https://github.com/cocoa-mhlw/cocoa/blob/b03037f5c2e21256e69b3eb291b6a68d66f0c076/Covid19Radar/Covid19Radar/Model/UserDataModel.cs#L16-L20
  • https://github.com/cocoa-mhlw/cocoa/blob/047b12bfc497e5e32bd96e24c2b5d76c8ca18bcf/Covid19Radar/Covid19Radar/Model/UserDataModel.cs#L8-L16

• PostRegisterUserAsync.HttpDataService

  • https://github.com/cocoa-mhlw/cocoa/blob/b03037f5c2e21256e69b3eb291b6a68d66f0c076/Covid19Radar/Covid19Radar/Services/HttpDataService.cs#L43-L69
  • https://github.com/cocoa-mhlw/cocoa/blob/047b12bfc497e5e32bd96e24c2b5d76c8ca18bcf/Covid19Radar/Covid19Radar/Services/HttpDataService.cs#L51-L72

• UploadSelfExposureKeysToServerAsync.UploadSelfExposureKeysToServerAsync

  • https://github.com/cocoa-mhlw/cocoa/blob/b03037f5c2e21256e69b3eb291b6a68d66f0c076/Covid19Radar/Covid19Radar/Services/ExposureNotificationHandler.cs#L222-L254
  • https://github.com/cocoa-mhlw/cocoa/blob/047b12bfc497e5e32bd96e24c2b5d76c8ca18bcf/Covid19Radar/Covid19Radar/Services/ExposureNotificationHandler.cs#L274-L347

• UploadSelfExposureKeysToServerAsync.CreateSubmissionAsync

  • https://github.com/cocoa-mhlw/cocoa/blob/b03037f5c2e21256e69b3eb291b6a68d66f0c076/Covid19Radar/Covid19Radar/Services/ExposureNotificationHandler.cs#L292-L302
  • https://github.com/cocoa-mhlw/cocoa/blob/047b12bfc497e5e32bd96e24c2b5d76c8ca18bcf/Covid19Radar/Covid19Radar/Services/ExposureNotificationHandler.cs#L381-L390

• UploadSelfExposureKeysToServerAsync.DiagnosisSubmissionParameter

  • https://github.com/cocoa-mhlw/cocoa/blob/b03037f5c2e21256e69b3eb291b6a68d66f0c076/Covid19Radar/Covid19Radar/Services/ExposureNotificationHandler.cs#L292-L302
  • https://github.com/cocoa-mhlw/cocoa/blob/047b12bfc497e5e32bd96e24c2b5d76c8ca18bcf/Covid19Radar/Covid19Radar/Services/ExposureNotificationHandler.cs#L381-L390

素人目に見ると、いずれの箇所でもv1.2.2ではUserUuidパラメーターは存在しておらず、当 Issue の懸念は取り除かれたように見えます。 再度のチェックをお願いしたく思います。

Purpose

<!-- Describe the intention of the changes being proposed. What problem does it solve or functionality does it add? -->

• #5 に対応。不具合対応を行う際に、どの機種でどのOSでどのバージョンで不具合が発生しているかを切り分けるためにiOSかAndroidかなどOSの情報とアプリのバージョンを表記する。GMS非対応の端末などは対応していないため切り分けのために必要。

Does this introduce a breaking change?

<!-- Mark one with an "x". -->

[ ] Yes
[x] No

Pull Request Type

What kind of change does this Pull Request introduce?

<!-- Please check the one that applies to this PR using "x". -->

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

How to Test

• Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

• Test the code <!-- Add steps to run the tests suite and/or manually test -->

What to Check

Verify that the following are valid

• アプリに関するお問い合わせを選択
• appsupport@cov19.mhlw.go.jpをタップ
• メールフォームを確認し、DEVICE_INFOが追記されていることを確認

DEVICE_INFO : APP_VERSION,Pixel 2 XL,(Google),Android,10
お名前：
ご連絡先：
お問い合わせ内容(カテゴリを次の中からお選びください)：1.アプリの仕組み、2.アプリの設定、 3.アプリの利用(通知など)、 4.その他
お問い合わせ本文：

Other Information

<!-- Add any other helpful information that may be needed here. -->

@takish @shinriyo ありがとうございます。 #5 にコメントした通り、v1.2.0で対応しているためこちらはクローズします。

Is your feature request related to a problem? Please describe. A clear and concise description of what the problem is. Ex. I'm always frustrated when [...] 不具合対応を行う際に、どの機種でどのOSでどのバージョンで不具合が発生しているかを切り分けるためにiOSかAndroidかなどOSの情報とアプリのバージョンをデフォルトで付与できるようにしたい。

Describe the solution you'd like A clear and concise description of what you want to happen. InqueryPageViewModelのBodyの本文の前に差し込まれるようにしたい

Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered. GMSに対応していない機種などもあるので、そのあたりの方からのお問い合わせが発生したときにシステムの不具合なのか、仕様上仕方がないことなのかに切り分ける必要がある。

Additional context Add any other context or screenshots about the feature request here.

• 現状

• 追加したいもの 以下の例のように本文との間に環境情報を追加する

@takish さん、遅くなりましたがPRもいただきありがとうございます。

いただいたご指摘も踏まえ、v1.2.0にて動作情報送信時にデバイス情報、アプリバージョンを取得するようにしました。

@tmurakami こちらご指摘ありがとうございます。今後対応します。

Is your feature request related to a problem? Please describe. Issue / Pull Request テンプレートが英語のみ。

Describe the solution you'd like Issue / Pull Request テンプレートが英語のみなので、日本語を併記する。Issue Template に Custom Template が残っているので削除。

Is your feature request related to a problem? Please describe.

CONTRIBUTING.md の中で、* waiting ラベルがついた Issue は・・・ とあるが、waiting-for-confirmation ラベルにしたので修正。

Is your feature request related to a problem? Please describe.

CONTRIBUTING.md の中で、* waiting ラベルがついた Issue は・・・ とあるが、waiting-for-confirmation ラベルにしたので修正。

あ、ビルドでの分岐を読み間違えてました。デバッグビルドで適用されるのはExposureNotification.OverrideNativeImplementationだけで、ExposureNotification.Init()は必ず実行されますね。

ご指摘の箇所についてはExposureNotificationの動作前に初期化する部分で、ここが実行されていないとExposureNotificationは正しく使えないと言う認識です。

たとえば、Androidのバックグラウンド処理ではExposureNotification. IsEnabledAsyncを呼び出しますが、ExposureNotification.Init()もしくはモック実装への入れ替えExposureNotification.OverrideNativeImplementation(new Services.TestNativeImplementation())を実行していなければ、このメソッドの戻り値は必ずfalseとなり、ExposureNotification.UpdateKeysFromServer()は実行されないと言う認識です。

https://github.com/cocoa-mhlw/cocoa/blob/d136940cccd7f096b0fc8f34172d8a9c293b2f8c/Covid19Radar/Xamarin.ExposureNotification/ExposureNotification.android.cs#L240-L241

Bumps pug-code-gen from 2.0.2 to 2.0.3. <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/pugjs/pug/commit/eedf9a3125aaea179dce36468401ef3bfc3db102"><code>eedf9a3</code></a> Don't ask escaping for boolean attributes at the beginning of attributes-list...</li> <li><a href="https://github.com/pugjs/pug/commit/6f38adf280242be79df276a9d998505c3bbab79a"><code>6f38adf</code></a> Add in additional resources of 2 packages (<a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2945">#2945</a>)</li> <li><a href="https://github.com/pugjs/pug/commit/d86ac52be4ec354cd6078797a80030ce3dc9e205"><code>d86ac52</code></a> new maintainer for pypugjs (<a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2947">#2947</a>)</li> <li><a href="https://github.com/pugjs/pug/commit/bd9e0de85b5975d1929887cc53722b02520a5597"><code>bd9e0de</code></a> [pug-lexer] Relax class name requirements (<a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2948">#2948</a>)</li> <li><a href="https://github.com/pugjs/pug/commit/af94a9f103785e7d6c46d64c069eb3f512ba6f9a"><code>af94a9f</code></a> <a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2887">#2887</a> Reorganize other languages section (<a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2928">#2928</a>)</li> <li><a href="https://github.com/pugjs/pug/commit/72fbf6a13b01e196bb8d0d42fd0c0104026263d1"><code>72fbf6a</code></a> Apply filters to includes in right-to-left order (<a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2932">#2932</a>)</li> <li><a href="https://github.com/pugjs/pug/commit/926f7c720112cac76cfedb003e25e9f43d3a1767"><code>926f7c7</code></a> small spelling errors fixed (<a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2885">#2885</a>)</li> <li><a href="https://github.com/pugjs/pug/commit/cb56c83850003457f36c093172a837a4ca16b230"><code>cb56c83</code></a> added pug template cache to other (<a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2868">#2868</a>)</li> <li><a href="https://github.com/pugjs/pug/commit/201941a7daa81f74e108f48f13b47d2af0d5c1b3"><code>201941a</code></a> fixes <a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2863">#2863</a> (<a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2864">#2864</a>)</li> <li><a href="https://github.com/pugjs/pug/commit/9cb96de9f32957ef2fc49a7cc2ba26f40becb2a4"><code>9cb96de</code></a> fixes <a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2866">#2866</a> (<a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2867">#2867</a>)</li> <li>Additional commits viewable in <a href="https://github.com/pugjs/pug/compare/pug-code-gen@2.0.2...pug@2.0.3">compare view</a></li> </ul> </details> <br />

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

<details> <summary>Dependabot commands and options</summary> <br />

You can trigger Dependabot actions by commenting on this PR:

• @dependabot rebase will rebase this PR
• @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
• @dependabot merge will merge this PR after your CI passes on it
• @dependabot squash and merge will squash and merge this PR after your CI passes on it
• @dependabot cancel merge will cancel a previously requested merge and block automerging
• @dependabot reopen will reopen this PR if it is closed
• @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
• @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
• @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
• @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
• @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
• @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language

You can disable automated security fix PRs for this repo from the Security Alerts page.

</details>

Bumps pug-code-gen from 2.0.2 to 2.0.3. <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/pugjs/pug/commit/eedf9a3125aaea179dce36468401ef3bfc3db102"><code>eedf9a3</code></a> Don't ask escaping for boolean attributes at the beginning of attributes-list...</li> <li><a href="https://github.com/pugjs/pug/commit/6f38adf280242be79df276a9d998505c3bbab79a"><code>6f38adf</code></a> Add in additional resources of 2 packages (<a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2945">#2945</a>)</li> <li><a href="https://github.com/pugjs/pug/commit/d86ac52be4ec354cd6078797a80030ce3dc9e205"><code>d86ac52</code></a> new maintainer for pypugjs (<a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2947">#2947</a>)</li> <li><a href="https://github.com/pugjs/pug/commit/bd9e0de85b5975d1929887cc53722b02520a5597"><code>bd9e0de</code></a> [pug-lexer] Relax class name requirements (<a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2948">#2948</a>)</li> <li><a href="https://github.com/pugjs/pug/commit/af94a9f103785e7d6c46d64c069eb3f512ba6f9a"><code>af94a9f</code></a> <a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2887">#2887</a> Reorganize other languages section (<a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2928">#2928</a>)</li> <li><a href="https://github.com/pugjs/pug/commit/72fbf6a13b01e196bb8d0d42fd0c0104026263d1"><code>72fbf6a</code></a> Apply filters to includes in right-to-left order (<a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2932">#2932</a>)</li> <li><a href="https://github.com/pugjs/pug/commit/926f7c720112cac76cfedb003e25e9f43d3a1767"><code>926f7c7</code></a> small spelling errors fixed (<a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2885">#2885</a>)</li> <li><a href="https://github.com/pugjs/pug/commit/cb56c83850003457f36c093172a837a4ca16b230"><code>cb56c83</code></a> added pug template cache to other (<a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2868">#2868</a>)</li> <li><a href="https://github.com/pugjs/pug/commit/201941a7daa81f74e108f48f13b47d2af0d5c1b3"><code>201941a</code></a> fixes <a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2863">#2863</a> (<a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2864">#2864</a>)</li> <li><a href="https://github.com/pugjs/pug/commit/9cb96de9f32957ef2fc49a7cc2ba26f40becb2a4"><code>9cb96de</code></a> fixes <a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2866">#2866</a> (<a href="https://github-redirect.dependabot.com/pugjs/pug/issues/2867">#2867</a>)</li> <li>Additional commits viewable in <a href="https://github.com/pugjs/pug/compare/pug-code-gen@2.0.2...pug@2.0.3">compare view</a></li> </ul> </details> <br />

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

<details> <summary>Dependabot commands and options</summary> <br />

You can trigger Dependabot actions by commenting on this PR:

• @dependabot rebase will rebase this PR
• @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
• @dependabot merge will merge this PR after your CI passes on it
• @dependabot squash and merge will squash and merge this PR after your CI passes on it
• @dependabot cancel merge will cancel a previously requested merge and block automerging
• @dependabot reopen will reopen this PR if it is closed
• @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
• @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
• @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
• @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
• @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
• @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language

You can disable automated security fix PRs for this repo from the Security Alerts page.

</details>