Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Generate shortlive token #437

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

CoCo-Japan-pan
Copy link
Contributor

fixes #436
環境変数DROPBOX_TOKENの代わりに、DROPBOX_REFRESH_TOKEN,DROPBOX_APP_KEY,DROPBOX_APP_SECRETを設定している場合は、これらを用いて一時的なアクセストークンを取得し、これをdropbox-tokenとして用いるようにしました。ドキュメントにも記述を加えました。
ご検討いただけると幸いです。

@CoCo-Japan-pan CoCo-Japan-pan force-pushed the generate-shortlive-token branch 2 times, most recently from 4d51d2b to 5f028cb Compare March 15, 2024 18:41
@CoCo-Japan-pan CoCo-Japan-pan marked this pull request as draft March 16, 2024 03:47
@CoCo-Japan-pan CoCo-Japan-pan force-pushed the generate-shortlive-token branch 5 times, most recently from 597b24a to e069dc6 Compare March 16, 2024 04:41
@CoCo-Japan-pan CoCo-Japan-pan marked this pull request as ready for review March 16, 2024 04:45
@CoCo-Japan-pan
Copy link
Contributor Author

CoCo-Japan-pan commented Mar 16, 2024

一時的なアクセストークンをrefresh tokenを用いて取得した場合、それはマスクされないので、これもマスクするようにしました。

@kzrnm
Copy link
Contributor

kzrnm commented Mar 16, 2024

マスクし直すより online-judge-tools/oj#917 をやった方が安全だと思いました。

cc. @koba-e964

@kzrnm
Copy link
Contributor

kzrnm commented Mar 16, 2024

online-judge-tools/oj#914

上記の PR とも関係しますが、リフレッシュトークンを使ってトークンを更新するのは online-judge-tools/oj の責務のようです。

@koba-e964

が妥当ではないかと思いました。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DROPBOX_TOKENの有効期限が短くなったので、毎回Refresh Tokenで再取得できると嬉しい
2 participants