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

Use $YUKICODER_TOKEN and $DROPBOX_TOKEN envvar for a security reason #917

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

Conversation

kzrnm
Copy link
Contributor

@kzrnm kzrnm commented Nov 1, 2022

#672

verify helper から yukicoder が利用できるとうれしいので対応したい。 実装は単に --yukicoder-token みたいなオプションを用意するのがよさそう。YUKICODER_TOKEN みたいな環境変数を設定させてそれを読むのは一般には自然だが、現状の online-judge-tools は他にはまったく環境変数を見ないので一貫性が損われる。

導入時は上記のような経緯でコマンドライン引数として実装されていました。

https://github.com/online-judge-tools/api-client/blob/7936b6c56ae0b056bdb9f73fff7c0681160c71de/onlinejudge_api/main.py#L29

しかし、その後に api-client ではセキュリティ的に良くないとコマンドライン引数は廃止されています。

全廃すると影響が多大なので「使えるようにはしているけど推奨はしない」くらいの扱いにしておくのが良いのではないかと思います。

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.

None yet

1 participant