Skip to content
This repository has been archived by the owner on Aug 31, 2022. It is now read-only.

認可処理を追加 #27

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kobayashi-m42
Copy link
Member

issueURL

#17

関連URL

[optional]

Doneの定義

スクリーンショット

[optional]

変更点概要

[required]

レビュアーに重点的にチェックして欲しい点

[optional]

補足情報

[optional]

@coveralls
Copy link

Pull Request Test Coverage Report for Build 394098447

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 46.707%

Totals Coverage Status
Change from base Build 391476345: 0.0%
Covered Lines: 78
Relevant Lines: 167

💛 - Coveralls

@kobayashi-m42 kobayashi-m42 self-assigned this Dec 6, 2020
@keitakn
Copy link
Member

keitakn commented Dec 6, 2020

案1

アカウントIDは必ず以下の形で送られる。

/v1/accounts/{account_id(UUIDv4形式)}

なので /v1/accounts/{account_id(UUIDv4形式)} から {account_id(UUIDv4形式)} を抜き出して、アカウントIDとアクセストークンのSubが一致するか確認する。

@keitakn
Copy link
Member

keitakn commented Dec 6, 2020


// アカウント取得
/v1/accounts/{account_id}

// クローゼットの中身全部
/v1/closets/accounts/{account_id}

// クローゼットの中身1個だけ
/v1/closets/{closet_id}/accounts/{account_id}

@keitakn
Copy link
Member

keitakn commented Dec 6, 2020

POST /v1/accounts のようなAPIが存在する。

これはアカウント作成時に呼ばれる奴。

kimono-appのDBにユーザーのアカウントID(CognitoSub)とメールアドレス、ニックネームが登録される可能性がある。

@keitakn
Copy link
Member

keitakn commented Dec 6, 2020

決定事項

  1. アカウントIDは常にアクセストークンから取得する
// アカウント取得
/v1/accounts

// クローゼットの中身全部
/v1/closets

// クローゼットの中身1個だけ
/v1/closets/{closet_id}
  1. URLのパスやパラメータにアカウントIDを付ける事は禁止
  2. アカウントID = Cognitoのsubを指す
  3. メールアドレスやニックネームはkimono-apiでは持たない、IDトークンの中身を参照

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.

None yet

3 participants