saito.k
|
0a970e814f
|
Merged PR 76: API実装（I/F）
## 概要
[Task1576: API実装（I/F）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1576)
- sas発行APIのIFを実装
- アップロード用のSAS発行API
- GET /blob/uploadSas
- ダウンロード用のSAS発行API
- GET /blob/downloadSas
- notification関連のフォーマット修正
- notification関連でlintエラーが出ていた箇所を修正
- openapi.jsonを生成するコマンドを使用できるように修正
## レビューポイント
- レスポンスとして返却する内容に不足は無いか
- URIはこれで良さそうか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルで確認
## 補足
- 相談、参考資料などがあれば
|
2023-04-14 01:08:32 +00:00 |
|
makabe
|
50cbcb1d81
|
メール認証URLのドメインを環境変数で設定
|
2023-04-11 17:50:39 +09:00 |
|
makabe.t
|
50f4cf5070
|
Merged PR 65: 画面実装（ユーザー認証画面）
## 概要
[Task1495: 画面実装（ユーザー認証画面）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1495)
- メールの認証URLから、認証を実行して結果を表示するまでの画面を実装しました。
- 認証画面
- 認証完了画面
- 成功
- 失敗
- 認証済み
- エラーハンドリング用のメソッドを`common`に追加しました。
- メールに送信される認証URLのパスを認証画面のパスに修正しました。
## レビューポイント
- エラーハンドリング用の処理は適切でしょうか？
- 改行を画面に対応させるために暫定の処置を入れています。対応に問題はないでしょうか。
- アカウント登録のメール送信について、パスを対象となる画面のパス`mail-confirm/`に変更しました。
- 対応として適切でしょうか？
## UIの変更
- [Task1495](https://ndstokyo.sharepoint.com/:f:/r/sites/Piranha/Shared%20Documents/General/OMDS/%E3%82%B9%E3%82%AF%E3%83%AA%E3%83%BC%E3%83%B3%E3%82%B7%E3%83%A7%E3%83%83%E3%83%88/Task1495?csf=1&web=1&e=bqT7nz)
## 動作確認状況
- ローカルで確認
|
2023-04-10 09:03:21 +00:00 |
|
saito.k
|
ee2e8dbd5d
|
Merged PR 64: デバイス登録API実装
## 概要
[Task1571: デバイス登録API実装](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1571)
- NotificationHubへのデバイス登録APIを実装
- NotificationHubへの接続
- 登録処理
## レビューポイント
- ディレクトリ構成に問題は無いか
## UIの変更
- Before/Afterのスクショなど
- スクショ置き場
## 動作確認状況
- ローカルでNotificationHubに接続できるところまで確認
## 補足
- 返却するエラーやサーバー側で設定する部分の登録情報については仮です。
- 「プロダクト バックログ項目 1389: デスクトップアプリ/モバイルアプリに様々な通知をしたい」で実装・設計する想定
|
2023-04-10 00:41:32 +00:00 |
|
makabe.t
|
0099614a5f
|
Merged PR 58: API実装（アカウント登録/Azure AD B2C）
## 概要
[Task1550: API実装（アカウント登録/Azure AD B2C）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1550)
- アカウント登録APIでAzure ADB2Cにユーザを登録する処理を追加しました。
- GraphAPIで登録処理をしています。
## レビューポイント
- ADB2Cへ登録する情報は認識通りか
- ADB2CへのGraphAPI接続のためにADB2Cテナントにアプリを追加しているが認証として問題ないか。
- 環境変数にアプリの情報を設定しています
- ADB2C_TENANT_ID=xxxxxxxx
- ADB2C_CLIENT_ID=xxxxxxxx
- ADB2C_CLIENT_SECRET=xxxxxxxx
## UIの変更
無し
## 動作確認状況
- ローカルで確認
- ADB2C、DBに設定項目が追加されていることを確認
|
2023-04-05 09:22:50 +00:00 |
|
湯本 開
|
9f5252baf8
|
Merged PR 56: API実装（アカウント登録）
## 概要
[Task1496: API実装（アカウント登録）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1496)
- アカウント登録のAzureAD B2Cへのユーザー作成部分以外について実装
- migration SQLに足りない要素があった箇所を修正(default追加)
- 各種環境変数を追加
- 秘密鍵/公開鍵を取得する方法を環境変数に変更（KeyVaultからWebAppsの機能で環境変数へ流し込む想定）
## レビューポイント
- 実装方法として問題がありそうな箇所が存在しないか
- 可読性の低い箇所が存在しないか
- Moduleの分け方、つなげ方などは問題ないか
- ラフスケッチと違い、Account作成と管理者ユーザー作成を同一トランザクションで行うよう修正したが問題ないか
## 動作確認状況
- メール送信以外はローカルで確認、メール送信部分は未確認
|
2023-03-29 03:55:44 +00:00 |
|
湯本 開
|
c1ed541d87
|
Merged PR 4: タスク 1385: API実装（認証/トークン発行）
## 概要
[Task 1385](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_workitems/edit/1385)
- IDトークンを受け取り、リフレッシュトークンとアクセストークンを発行するAPIを実装
- リフレッシュトークンを受け取り、アクセストークンを発行するAPIを実装
- アクセストークン、リフレッシュトークンの中身は中身を使用するPBIが存在しないので仮のもの
## レビューポイント
- 本PBIで必要な範囲の機能を十分持っているか
- コードに不足部分、問題点などはなさそうか
- テスト項目は妥当か
- アクセストークン、リフレッシュトークンの中身はまだ仮で問題ないか
## 動作確認状況
- ローカルで動作確認
|
2023-03-07 23:35:53 +00:00 |
|
makabe.t
|
41f0213fe9
|
Merged PR 1: タスク 1359: API実装（認証/IDトークン検証）
## 概要
[タスク 1359: API実装（認証/IDトークン検証）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/OMDSDictation/_workitems/edit/1359)
- IDトークンを受け取って内容を検証し、デコードしたペイロードを返すサービスを実装しました。
## レビューポイント
- サービスの処理の流れが認識とあっているか。
- ADB2CのAPI呼び出しを別サービスにしているが問題ないか
- 公開鍵の変換処理を別サービスに切り出しているが構成に問題はないか。
- トークンの検証をエラーごとに処理しているがエラー内容は認識通りか
## UIの変更
- なし
## 動作確認状況
- テストが通ることを確認
# 備考
- IDトークンを検証して中身を返すまでの実装です。
|
2023-03-07 01:30:13 +00:00 |
|