案）トークン保存ファイルについて - Githubissues

YUKAI / emo-platform-api-python

MIT License

3 stars 0 forks source link

案）トークン保存ファイルについて #12

Closed SebMiller closed 3 years ago

SebMiller commented 3 years ago

現状

初期値はOS環境レベルで指定が必要
トークンが固定のファイルに保存される
トークンアップデートが必要場合は自動で行って、ファイルに保存する

懸念点

トークンがライブラリーに紐付いてるので
- インスタンスを増やして同時に複数のアカウントを使えない
- 別々のプログラムから別々のアカウントを使えない（pipenvとかを使わない場合）
ユーザーがトークンの保存場所をコントロール出来ない

案

トークンの初期値をClientの __init__ で渡せるようにする
トークンの保存場所をClientの __init__ で渡せるようにする（個人的に ii. が良さそう） 1) 保存用のファイルパスを渡す or 2) トークンアップデート時用のcallbackを渡す(callbackのパラリンピックは新トークン)、保存自体は開発者に任せる

KeitaIto123 commented 3 years ago

レビューありがとうございます
同時に複数のアカウントを使えない点について
- 1つのアカウントに複数のemo(ルーム)を紐づけられることから、複数のアカウントを使用したい場面が想定できないので、問題ないかと思います。
- どうしてもやりたい場合は、pipenvとかを使ってもらう形にします
ユーザーがトークンの保存場所をコントロール出来ない点について
- 確かに不便ですので、提案された形のいずれかあるいは両方を実装しようかと思います

KeitaIto123 commented 3 years ago

20 にてclientの初期化時にトークンの保存場所を指定できるようにしました